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-14054: [C++][Docs] Simplify C++ row conversion example #11197
Conversation
bkmgit
commented
Sep 21, 2021
- Add explanations for code example
- Print out entries
- make variable names clearer
|
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.
Thanks for doing this. Some comments.
@amol- Since you're getting acquainted with Arrow C++, what do you think about this? |
Thanks for the feedback. Will incorporate this. |
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.
Thanks for the update. A couple more comments below.
- Add explanations for code example - Print out entries - make variable names clearer
07725be
to
1b47327
Compare
- Add explanations for code example - Print out entries - make variable names clearer Closes apache#11197 from bkmgit/bkmgit-row-conversion Lead-authored-by: Benson Muite <bkmgit@users.noreply.github.com> Co-authored-by: Antoine Pitrou <antoine@python.org> Signed-off-by: Antoine Pitrou <antoine@python.org>