Xianqing He has posted comments on this change. ( http://gerrit.cloudera.org:8080/19046 )
Change subject: IMPALA-11558: Ensure one Kudu client created (FE) for the specified Kudu master addresses ...................................................................... Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/19046/5/fe/src/test/java/org/apache/impala/util/KuduUtilTest.java File fe/src/test/java/org/apache/impala/util/KuduUtilTest.java: http://gerrit.cloudera.org:8080/#/c/19046/5/fe/src/test/java/org/apache/impala/util/KuduUtilTest.java@57 PS5, Line 57: int concurrent = 5; > Hi xianqing, I think exposing the mutable map in patch 6 isn't a good pract Done -- To view, visit http://gerrit.cloudera.org:8080/19046 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1003556d3afc8e8216142cac4007a4c99046caeb Gerrit-Change-Number: 19046 Gerrit-PatchSet: 7 Gerrit-Owner: Xianqing He <hexianqing...@126.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Jian Zhang <zjsar...@gmail.com> Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com> Gerrit-Reviewer: Xiang Yang <yx91...@126.com> Gerrit-Reviewer: Xianqing He <hexianqing...@126.com> Gerrit-Comment-Date: Sun, 09 Oct 2022 09:01:12 +0000 Gerrit-HasComments: Yes