yanlin-Lynn opened a new pull request #1442: [CALCITE-3319] Remove throwing 
AssertionError code in ReduceDecimalsRule
URL: https://github.com/apache/calcite/pull/1442
 
 
   There exist code that throws AssertionError in ReduceDecimalsRule every time 
it was ran to.
   
   > List<RexNode> newOperands = apply(call.getOperands());
         if (true) {
           // FIXME: Operands are now immutable. Create a new call with
           //   new operands?
           throw new AssertionError();
         }
   
   This PR tries to remove this part of code, and add unit test case for the 
new code path.
   Please refer to 
[CALCITE-3319](https://issues.apache.org/jira/browse/CALCITE-3319) for full 
stack trace.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to