jorisvandenbossche commented on PR #51372: URL: https://github.com/apache/arrow/pull/51372#issuecomment-6082276676
BTW, it would help with reviewing if you left your own further code changes as separate commits, instead of combining it within the rebased commits from the other PR (now it is hard to see which review comments from https://github.com/apache/arrow/pull/50409/ you have addressed) > Also included, found while going through the files: a shadowed variable in AppendUTF32, a missing extension path in the MonthDayNano conversion, and the MonthDayNano docstring. What is this exactly about? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
