jorisvandenbossche commented on pull request #8805: URL: https://github.com/apache/arrow/pull/8805#issuecomment-754839607
I didn't yet update it regarding the discussion on the keyword arguments API above at https://github.com/apache/arrow/pull/8805#discussion_r542440135 We probably want to support the current "verbose" way of writing the options anyway? In which case it might be fine to defer a more ergonomic API to a follow-up JIRA (since this needs to get in this week) Alternatively, I could add an `order` keyword on the python side (as mentioned in https://github.com/apache/arrow/pull/8805#discussion_r542510825), and then there translate something like `sort_keys=["col1", "col2"], order="descending"` to `sort_keys=[("col1", "descending"), ("col2", "descending")]` passed to the C++ options. In that case, you only need to full verbose API when wanting to sort with multiple columns with a different ascending/descending, and thus might already help for many common cases. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
