Jason Fehr has posted comments on this change. ( http://gerrit.cloudera.org:8080/21048 )
Change subject: IMPALA-12426: Workload Management Supporting Changes ...................................................................... Patch Set 13: (2 comments) http://gerrit.cloudera.org:8080/#/c/21048/12/be/src/util/network-util.h File be/src/util/network-util.h: http://gerrit.cloudera.org:8080/#/c/21048/12/be/src/util/network-util.h@103 PS12, Line 103: /// by port (numerically) and finally by uds address (alphabetically). > This description is no longer accurate. Now goes host > port > uds. Done http://gerrit.cloudera.org:8080/#/c/21048/12/be/src/util/string-util.h File be/src/util/string-util.h: http://gerrit.cloudera.org:8080/#/c/21048/12/be/src/util/string-util.h@99 PS12, Line 99: /// enable comma delimited string building where the last instance of the comma needs to > typo: delimited Done -- To view, visit http://gerrit.cloudera.org:8080/21048 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee23334ec56a18b192a75d052468bf59159b6424 Gerrit-Change-Number: 21048 Gerrit-PatchSet: 13 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Mon, 26 Feb 2024 18:44:27 +0000 Gerrit-HasComments: Yes
