Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23016 )
Change subject: [server] KUDU-1457 [2/n] Add support for IPv6 ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/23016/5//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/23016/5//COMMIT_MSG@7 PS5, Line 7: [server] KUDU-1457 [2/n] Add support for IPv6 A couple of nits for your consideration w.r.t. the description of this and other patches in the context of KUDU-1457 * it's already enough to have KUDU-1457 as an umbrella tag, so [server] might be omitted and replaced with actual summary (see the next item) * since KUDU-1457 tag is already present, instead of repeating 'Add support for IPv6' in every commit, maybe add a summary of the actual modifications in every particular changelist, e.g. for this patch it might be 'add IPv6 support for squeasel' These are just nits, though. -- To view, visit http://gerrit.cloudera.org:8080/23016 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee175a00a7ac9e20a31ddb8396468cfcd8f3b6f9 Gerrit-Change-Number: 23016 Gerrit-PatchSet: 5 Gerrit-Owner: Ashwani Raina <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Comment-Date: Fri, 18 Jul 2025 02:59:06 +0000 Gerrit-HasComments: Yes
