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

Mon, 24 Oct 2022 05:08:33 -0700 


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-unsubscr...@calcite.apache.org

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

Reply via email to