Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/20810 )
Change subject: WIP: Skip KRPC in local exchanges ...................................................................... Patch Set 7: (12 comments) http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/krpc-data-stream-recvr.cc File be/src/runtime/krpc-data-stream-recvr.cc: http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/krpc-data-stream-recvr.cc@343 PS7, Line 343: if (payload->rpc_context != nullptr) TRACE_TO(payload->rpc_context->trace(), "Enqueuing deferred RPC"); line too long (105 > 90) http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/krpc-data-stream-recvr.cc@493 PS7, Line 493: //TRACE_TO(rpc_context->trace(), "Failed to deserialize batch: $0", status.GetDetail()); line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/krpc-data-stream-recvr.cc@647 PS7, Line 647: if (ctx->rpc_context != nullptr) TRACE_TO(ctx->rpc_context->trace(), "Batch queue is full"); line too long (99 > 90) http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/krpc-data-stream-recvr.cc@670 PS7, Line 670: if (ctx->rpc_context != nullptr) recvr_->deferred_rpc_tracker()->Release(ctx->rpc_context->GetTransferSize()); line too long (114 > 90) http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/krpc-data-stream-recvr.cc@675 PS7, Line 675: if (ctx->rpc_context != nullptr) DataStreamService::RespondRpc(status, ctx->response, ctx->rpc_context); line too long (106 > 90) http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/krpc-data-stream-sender.cc File be/src/runtime/krpc-data-stream-sender.cc: http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/krpc-data-stream-sender.cc@695 PS7, Line 695: RETURN_IF_ERROR(parent_->SerializeBatch(batch, serialization_batch->get(), !is_local_)); line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/local-row-batch-channel.h File be/src/runtime/local-row-batch-channel.h: http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/local-row-batch-channel.h@38 PS7, Line 38: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/local-row-batch-channel.cc File be/src/runtime/local-row-batch-channel.cc: http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/local-row-batch-channel.cc@50 PS7, Line 50: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/local-row-batch-channel.cc@84 PS7, Line 84: } line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/local-row-batch-channel.cc@106 PS7, Line 106: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/local-row-batch-channel.cc@173 PS7, Line 173: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/20810/7/be/src/runtime/local-row-batch-channel.cc@182 PS7, Line 182: Status LocalRowBatchChannelManager::RegisterSender(RuntimeState* state, line has trailing whitespace -- To view, visit http://gerrit.cloudera.org:8080/20810 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee16cd943d76d04874bce7f3959e74b4685adb6e Gerrit-Change-Number: 20810 Gerrit-PatchSet: 7 Gerrit-Owner: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Mon, 18 Dec 2023 09:33:45 +0000 Gerrit-HasComments: Yes
