ianmcook commented on pull request #11212: URL: https://github.com/apache/arrow/pull/11212#issuecomment-925333573
LGTM, thank you! I'm curious why use a `NullPlacement` enum to instead of just a `nulls_last` Boolean. Might the enum be extended to include other options in the future (for example controlling whether NaNs come before or after nulls)? -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
