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

Follow ups for #206 #207

Closed
2 tasks
eddyxu opened this issue Sep 27, 2022 · 0 comments
Closed
2 tasks

Follow ups for #206 #207

eddyxu opened this issue Sep 27, 2022 · 0 comments
Assignees
@eddyxu
Labels
arrow Apache Arrow related issues bug Something isn't working ci Github Action or Test issues

Comments

@eddyxu
Copy link
Contributor

eddyxu commented Sep 27, 2022 •
edited
Loading

  • Add appropriate test case
  • Refactor ScanBatch into class and setting appropriate access to its members.
@eddyxu eddyxu added bug Something isn't working ci Github Action or Test issues arrow Apache Arrow related issues labels Sep 27, 2022
@eddyxu eddyxu self-assigned this Sep 27, 2022
@eddyxu eddyxu changed the title Add test case for #206 Follow ups for #206 Sep 27, 2022
@eddyxu eddyxu mentioned this issue Sep 27, 2022
Merged
@eddyxu eddyxu closed this as completed Jan 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@eddyxu eddyxu

Labels
arrow Apache Arrow related issues bug Something isn't working ci Github Action or Test issues
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@eddyxu