Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/23686 )
Change subject: IMPALA-14408 ADDENDUM: Log Calcite exception in profile ...................................................................... Patch Set 1: Code-Review+1 (1 comment) This makes sense to me. We can revisit exactly how much we need to include for CalciteFailureReason, but this unblocks us for now. We may end up including the exception message with some sort of handling for known-long messages (e.g. skip the exception message for ParseException). http://gerrit.cloudera.org:8080/#/c/23686/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/23686/1//COMMIT_MSG@12 PS1, Line 12: Change-Id: Ia18a52c488f9c73d51690997b277fd8e918c645f This needs to move down to the bottom -- To view, visit http://gerrit.cloudera.org:8080/23686 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia18a52c488f9c73d51690997b277fd8e918c645f Gerrit-Change-Number: 23686 Gerrit-PatchSet: 1 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Pranav Lodha <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Wed, 19 Nov 2025 23:12:39 +0000 Gerrit-HasComments: Yes
