You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2019/10/24 02:54:03 UTC

[GitHub] [calcite] danny0405 opened a new pull request #1527: [CALCITE-3441] Remove SqlTypeExplicitPrecedenceList.COMPACT_NUMERIC_T…

danny0405 opened a new pull request #1527: [CALCITE-3441] Remove SqlTypeExplicitPrecedenceList.COMPACT_NUMERIC_T…
URL: https://github.com/apache/calcite/pull/1527
 
 
   …YPES because the NULL delimiters are useless
   
   We do not need the "null" value as delimiter for the equivalence class
   because method Util.skip already did that.
   
   This also tweak the "needToCast" check for implicit type coercion:
   change the "canCast" check into an assert, that means, we should not
   swallow the cast when the coercion is not allowed in the type coercion matrix.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services