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.
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
feat: add Arrow IPC output format for http rest api #3177
feat: add Arrow IPC output format for http rest api #3177
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3177 +/- ##
==========================================
- Coverage 85.39% 85.30% -0.10%
==========================================
Files 827 841 +14
Lines 135264 137641 +2377
==========================================
+ Hits 115509 117411 +1902
- Misses 19755 20230 +475 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I hereby agree to the terms of the GreptimeDB CLA
This patch is initial work to add arrow output format for our http sql api. By using arrow output format it is possible to do zero-copy with arrowjs ecosystem, including visualization library vega (with vega-arrow-loader) and many more.
I will be seeking to add streaming support for output types like csv and arrow in future patch.
Checklist
Refer to a related PR or issue link (optional)