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 2022/10/24 12:08:27 UTC

[GitHub] [calcite] wojustme commented on pull request #2686: [CALCITE-4982] NonNull field shouldn't be pushed down into leaf of outer-join in 'ProjectJoinTransposeRule'

wojustme commented on PR #2686:
URL: https://github.com/apache/calcite/pull/2686#issuecomment-1288937219

   In my opinion, this case only works on in CALCITE's optimizer or validator, which some expresion only change nullable attribute.
   we couldn't write an sql `cast(c1 as not integer)`, and we only write an sql `cast(c1 as integer)`.
   
   This pr will fix the issue with a little modification. Maybe we get a good solution from the discussion of [CALCITE-5315](https://issues.apache.org/jira/browse/CALCITE-5315).
   


-- 
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: commits-unsubscribe@calcite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org