Aman Sinha has posted comments on this change. ( http://gerrit.cloudera.org:8080/14690 )
Change subject: IMPALA-9146: Add a configurable limit for the size of broadcast input. ...................................................................... Patch Set 6: (2 comments) Thanks for the code review. I have addressed the review comments. For the unit tests, I ended up creating separate test files and changing the option through the Java code. I see some tests are setting QUERYOPTIONS within a single test file..not sure if there's a preference. http://gerrit.cloudera.org:8080/#/c/14690/4//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/14690/4//COMMIT_MSG@30 PS4, Line 30: - Added a few unit testis in PlannerTest that (a) set the > Can you also add the option to query-options-test just to test the validati Done http://gerrit.cloudera.org:8080/#/c/14690/4/testdata/workloads/functional-planner/queries/PlannerTest/broadcast-bytes-limit.test File testdata/workloads/functional-planner/queries/PlannerTest/broadcast-bytes-limit.test: PS4: > Ah, sure will add that. Done -- To view, visit http://gerrit.cloudera.org:8080/14690 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ibe5639ca38acb72e0194aa80bc6ebb6cafb2acd9 Gerrit-Change-Number: 14690 Gerrit-PatchSet: 6 Gerrit-Owner: Aman Sinha <amsi...@cloudera.com> Gerrit-Reviewer: Aman Sinha <amsi...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Comment-Date: Wed, 20 Nov 2019 16:56:07 +0000 Gerrit-HasComments: Yes