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
ARROW-4734: [Go] Add option to write a header for CSV writer #3866
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3866 +/- ##
==========================================
- Coverage 87.78% 87.58% -0.21%
==========================================
Files 727 653 -74
Lines 87892 84092 -3800
Branches 1252 0 -1252
==========================================
- Hits 77158 73653 -3505
+ Misses 10616 10325 -291
+ Partials 118 114 -4
Continue to review full report at Codecov.
|
@sbinet I update the PR as what we discussed, can you help to take a look? |
@sbinet ping |
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.
last nit-picks and then LGTM.
@sbinet