Yida Wu has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22724 )

Change subject: IMPALA-13925: Deflake TestFrontendConnectionLimit
......................................................................


Patch Set 4: Code-Review+1

(6 comments)

LGTM, just a few minor nits in the commit message

http://gerrit.cloudera.org:8080/#/c/22724/4//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22724/4//COMMIT_MSG@10
PS4, Line 10: pick
nit. picks


http://gerrit.cloudera.org:8080/#/c/22724/4//COMMIT_MSG@10
PS4, Line 10: client
nit. clients


http://gerrit.cloudera.org:8080/#/c/22724/4//COMMIT_MSG@11
PS4, Line 11: limit
nit. limits


http://gerrit.cloudera.org:8080/#/c/22724/4//COMMIT_MSG@13
PS4, Line 13: is
remove "is"?


http://gerrit.cloudera.org:8080/#/c/22724/4//COMMIT_MSG@13
PS4, Line 13: able to
nit. be able to


http://gerrit.cloudera.org:8080/#/c/22724/4//COMMIT_MSG@18
PS4, Line 18: skip creating
nit. skipping the creation of



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I53c6760f5e3734397746b5a228345c9df38eabcb
Gerrit-Change-Number: 22724
Gerrit-PatchSet: 4
Gerrit-Owner: Riza Suminto <riza.sumi...@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>
Gerrit-Reviewer: Yida Wu <wydbaggio...@gmail.com>
Gerrit-Comment-Date: Thu, 03 Apr 2025 22:21:57 +0000
Gerrit-HasComments: Yes

Reply via email to