pfzhan commented on PR #3326: URL: https://github.com/apache/calcite/pull/3326#issuecomment-1740819737
> Overall the changes LGTM, however when I apply locally the patch for [CALCITE-5923](https://issues.apache.org/jira/browse/CALCITE-5923), which makes all the tests under SqlOperatorTest run as expected, I get some errors. > > ``` > ./gradlew cleanTest :core:test --tests CalciteSqlOperatorTest > ``` > > @pfzhan Can you please check if those errors are related with the changes here? I pulled the latest code locally and ran this command and it was OK. BUILD SUCCESSFUL in 56s 74 actionable tasks: 19 executed, 3 from cache, 52 up-to-date The remote build cache was disabled during the build due to errors. (base) ➜ calcite git:(CALCITE-5860) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
