beliefer commented on PR #43786: URL: https://github.com/apache/spark/pull/43786#issuecomment-1809870537
> @beliefer, are you referring to the `isSortAsc` argument of the mode function or are you referring to the new `MODE() WITHIN GROUP (ORDER BY col [ASC|DESC])` syntax by "mainstream databases support"? Because I want add the new `MODE() WITHIN GROUP (ORDER BY col [ASC|DESC])` syntax by "mainstream databases supported. Please see https://github.com/apache/spark/pull/43663. I found that the current implementation of `mode` only supports sort by column `ASC` if we make `deterministicExpr` is true. We should let `mode` also supports by column `DESC` -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
