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 <k...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com>
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Michael Ho <k...@cloudera.com>
Gerrit-Reviewer: Mostafa Mokhtar <mmokh...@cloudera.com>

Reply via email to