[GitHub] [kylin] zznlime commented on pull request #2001: KYLIN-5279 Default null collation for Kylin5.0 sqls should be null-last for ASC order and null-first for DESC order.

Tue, 25 Oct 2022 06:32:52 -0700 


zznlime commented on PR #2001:
URL: https://github.com/apache/kylin/pull/2001#issuecomment-1290563309

   Thanks for the suggestion! Have remove the config 
'kylin.query.asc-order-null-first-enable', and reuse 
KylinConfigBase#getCalciteExtrasProperties for setting default null collation.
   
   @hit-lacus 


-- 
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]

Reply via email to