Ashwani Raina has posted comments on this change. ( http://gerrit.cloudera.org:8080/19500 )
Change subject: [subprocess] better handling of oversized messages ...................................................................... Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/19500/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19500/1//COMMIT_MSG@7 PS1, Line 7: better handling of oversized messages Out of curiosity, is this patch added in context of a known issue that exhibited case of oversized messages? http://gerrit.cloudera.org:8080/#/c/19500/1//COMMIT_MSG@21 PS1, Line 21: server not just bails nit: server does not just bail http://gerrit.cloudera.org:8080/#/c/19500/1//COMMIT_MSG@22 PS1, Line 22: but nit: rather http://gerrit.cloudera.org:8080/#/c/19500/1//COMMIT_MSG@23 PS1, Line 23: discard of the message nit: discard the message http://gerrit.cloudera.org:8080/#/c/19500/1//COMMIT_MSG@26 PS1, Line 26: reflect on nit: reflect -- To view, visit http://gerrit.cloudera.org:8080/19500 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I05b09e757f304b22e37438c2445ecc161ef412c9 Gerrit-Change-Number: 19500 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 15 Feb 2023 12:27:19 +0000 Gerrit-HasComments: Yes
