olivrlee opened a new pull request, #3715:
URL: https://github.com/apache/calcite/pull/3715

   … serializing toRex
   
   Uncovered a bug in RelJson#toRex for the TIMESTAMP_DIFF call for Big Query 
dialect which is translated to Datetime MINUS. MINUS_DATE uses an ARG2_NULLABLE 
return type inference which requires 3 operands, however there are only 2 
operands for BQ. The solution is to add in "type" when serializing to JSON in 
RelJson.toJson(RexNode node) [like we do for CAST] so that jsonType will be 
defined in toRex and it doesn't rely on the operands.
   
   Co-authored-by: TJ Banghart <tjbangh...@gmail.com>


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