Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

int-676 Introduce a ParsedRow class for parsing result (including the row number) + expose the stream_parse method #28

Merged
merged 2 commits into from
May 17, 2024
Merged

int-676 Introduce a ParsedRow class for parsing result (including the row number) + expose the stream_parse method #28

merged 2 commits into from
May 17, 2024

Conversation

antoine-b-smartway
Copy link
Contributor

The need was:

  • to expose the Schema stream_parse method in the same manner than parse
  • return the row number associated to the data in the stream_parse method -> creation of a class ParsedRow to ease the usage of the result of the method

@antoine-b-smartway
int-676 Introduce a ParsedRow class for parsing result (including the…
5e96299 
… row number) + expose the stream_parse method
@antoine-b-smartway antoine-b-smartway self-assigned this May 17, 2024
@antoine-b-smartway antoine-b-smartway merged commit cf384d9 into main May 17, 2024
5 checks passed
@antoine-b-smartway antoine-b-smartway deleted the task/int-676-add-row-number-in-parsing-result branch May 17, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ducdetronquito ducdetronquito ducdetronquito approved these changes

@sGeeK44 sGeeK44 sGeeK44 approved these changes

@EwenBALOUIN EwenBALOUIN Awaiting requested review from EwenBALOUIN

Assignees

@antoine-b-smartway antoine-b-smartway

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@antoine-b-smartway @sGeeK44 @ducdetronquito