Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/16057 )
Change subject: IMPALA-9382: part 2/3: aggregate profiles sent to coordinator ...................................................................... Patch Set 13: (3 comments) http://gerrit.cloudera.org:8080/#/c/16057/13/be/src/runtime/coordinator-backend-state.cc File be/src/runtime/coordinator-backend-state.cc: http://gerrit.cloudera.org:8080/#/c/16057/13/be/src/runtime/coordinator-backend-state.cc@823 PS13, Line 823: // Merge this instance profile (which may or may not included the full instance profile line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/16057/13/be/src/runtime/coordinator-backend-state.cc@824 PS13, Line 824: // from the executor) into the aggregated profile. We need to do this regardless of the line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/16057/13/be/src/util/runtime-profile.cc File be/src/util/runtime-profile.cc: http://gerrit.cloudera.org:8080/#/c/16057/13/be/src/util/runtime-profile.cc@539 PS13, Line 539: DCHECK_LT(*node_idx, src.nodes.size()); const TRuntimeProfileNode& node = src.nodes[*node_idx]; line too long (98 > 90) -- To view, visit http://gerrit.cloudera.org:8080/16057 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic680cbfe94c939c2a8fad9d0943034ed058c6bca Gerrit-Change-Number: 16057 Gerrit-PatchSet: 13 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Fri, 13 Nov 2020 05:00:23 +0000 Gerrit-HasComments: Yes
