westonpace commented on PR #34137: URL: https://github.com/apache/arrow/pull/34137#issuecomment-1435355922
@lidavidm Sorry, I requested your review a bit too early. I started working on the order by node and realized that the ordering I had in place didn't support the ability to specify null placement. So I changed it to a proper class similar to SortNodeOptions. The change is pretty minor but I'll leave up unless you want to take another look. -- 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]
