Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/23008 )

Change subject: IMPALA-13649: Fix stack overflow in TestExprDepthLimit with 
JDK17
......................................................................

IMPALA-13649: Fix stack overflow in TestExprDepthLimit with JDK17

TestExprDepthLimit hit stack overflow when composing expression tree 10x
deeper than Expr.EXPR_DEPTH_LIMIT (1000). The test choose 10x for
negative test where expression tree is too deep. This patch lower the
multiple to 8x so that the expression tree does not hit stack overflow
during creation.

Testing:
Pass FE_TEST with JDK17.

Change-Id: Ibbe31598600da4b7f1510542ff9f32c8e8e1f9ad
Reviewed-on: http://gerrit.cloudera.org:8080/23008
Reviewed-by: Csaba Ringhofer <csringho...@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
---
M fe/src/test/java/org/apache/impala/analysis/AnalyzeExprsTest.java
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Csaba Ringhofer: Looks good to me, approved
  Impala Public Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/23008
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibbe31598600da4b7f1510542ff9f32c8e8e1f9ad
Gerrit-Change-Number: 23008
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>

Reply via email to