Hello Lars Volker, Mostafa Mokhtar, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9690
to look at the new patch set (#4).
Change subject: IMPALA-6685: Improve profiles in KrpcDataStreamRecvr and
KrpcDataStreamSender
......................................................................
IMPALA-6685: Improve profiles in KrpcDataStreamRecvr and KrpcDataStreamSender
This change implements a couple of improvements to the profiles of
KrpcDataStreamRecvr and KrpcDataStreamSender:
- track pending number of deferred row batches over time in KrpcDataStreamRecvr
- track the number of bytes dequeued over time in KrpcDataStreamRecvr
- track the min/max/avg of time row batches spent in deferred queue
- track the number of bytes sent from KrpcDataStreamSender over time
Sample profile of an Exchange node instance:
EXCHANGE_NODE (id=82):(Total: 1s367ms, non-child: 418.734us, %
non-child: 0.03%)
- ConvertRowBatchTime: 229.386us
- PeakMemoryUsage: 54.24 KB (55541)
- RowsReturned: 1.80K (1800)
- RowsReturnedRate: 1.32 K/sec
Buffer pool:
- AllocTime: 8.537us
- CumulativeAllocationBytes: 48.00 KB (49152)
- CumulativeAllocations: 4 (4)
- PeakReservation: 48.00 KB (49152)
- PeakUnpinnedBytes: 0
- PeakUsedReservation: 48.00 KB (49152)
- ReadIoBytes: 0
- ReadIoOps: 0 (0)
- ReadIoWaitTime: 0.000ns
- WriteIoBytes: 0
- WriteIoOps: 0 (0)
- WriteIoWaitTime: 0.000ns
Dequeue:
BytesDequeued(500.000ms): 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 36.91 KB, 36.91 KB, 36.91 KB
- FirstBatchWaitTime: 0.000ns
- TotalBytesDequeued: 36.91 KB (37800)
- TotalGetBatchTime: 1s366ms
- DataWaitTime: 1s366ms
Enqueue:
BytesReceived(500.000ms): 0, 0, 0, 0, 0, 0, 0, 8.10 KB, 14.24 KB,
14.24 KB, 14.24 KB, 14.24 KB, 14.24 KB, 14.24 KB, 14.24 KB, 14.24 KB, 14.24 KB,
14.24 KB, 14.24 KB, 14.24 KB, 14.24 KB, 14.24 KB, 14.24 KB, 14.24 KB, 14.24 KB,
14.24 KB, 14.24 KB
DeferredQueueSize(500.000ms): 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0, 0, 0
- DispatchTime: (Avg: 258.755us ; Min: 193.825us ; Max:
323.688us ; Number of samples: 2)
- DeserializeRowBatchTime: 60.318us
- TotalBatchesEnqueued: 2 (2)
- TotalBatchesReceived: 2 (2)
- TotalBytesReceived: 14.24 KB (14579)
- TotalEarlySenders: 0 (0)
- TotalEosReceived: 1 (1)
- TotalRPCsDeferralTime: 8s171ms
- TotalRPCsDeferred: 1 (1)
Sample sender's profile:
KrpcDataStreamSender (dst_id=82):(Total: 9s605ms, non-child: 125.726ms,
% non-child: 1.31%)
BytesSent(500.000ms): 0, 0, 0, 0, 24.29 KB, 24.29 KB, 24.29 KB, 24.29
KB, 24.29 KB, 24.29 KB, 24.29 KB, 24.29 KB, 24.29 KB, 24.29 KB, 24.29 KB, 24.29
KB, 24.29 KB, 24.29 KB, 24.29 KB, 24.29 KB, 24.29 KB, 30.43 KB, 30.43 KB, 30.43
KB
- EosSent: 3 (3)
- NetworkThroughput: 290.09 KB/sec
- PeakMemoryUsage: 29.53 KB (30240)
- RowsReturned: 1.80K (1800)
- RpcFailure: 0 (0)
- RpcRetry: 0 (0)
- SerializeBatchTime: 649.087us
- TotalBytesSent: 42.71 KB (43737)
- UncompressedRowBatchSize: 110.74 KB (113400)
Change-Id: I8ba405921b3df920c1e85b940ce9c8d02fc647cd
---
M be/src/kudu/rpc/rpc_context.cc
M be/src/kudu/rpc/rpc_context.h
M be/src/runtime/data-stream-test.cc
M be/src/runtime/krpc-data-stream-mgr.cc
M be/src/runtime/krpc-data-stream-recvr.cc
M be/src/runtime/krpc-data-stream-recvr.h
M be/src/runtime/krpc-data-stream-sender.cc
M be/src/runtime/krpc-data-stream-sender.h
M be/src/runtime/runtime-state.h
M be/src/service/data-stream-service.cc
M be/src/service/data-stream-service.h
M common/protobuf/data_stream_service.proto
12 files changed, 282 insertions(+), 139 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/90/9690/4
--
To view, visit http://gerrit.cloudera.org:8080/9690
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8ba405921b3df920c1e85b940ce9c8d02fc647cd
Gerrit-Change-Number: 9690
Gerrit-PatchSet: 4
Gerrit-Owner: Michael Ho <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Reviewer: Michael Ho <[email protected]>
Gerrit-Reviewer: Mostafa Mokhtar <[email protected]>
