Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/19897
Change subject: IMPALA-10180: Add summary stats for client fetch wait time ...................................................................... IMPALA-10180: Add summary stats for client fetch wait time This adds ClientFetchWaitTimeStats to the runtime profile to track the min/max/# of samples for ClientFetchWaitTimer. Here is some sample output: - ClientFetchWaitTimeStats: (Avg: 161.554ms ; Min: 101.411ms ; Max: 461.728ms ; Number of samples: 6) - ClientFetchWaitTimer: 969.326ms This also fixes the definition of ClientFetchWaitTimer to avoid including time after end of fetch. When the client is closing the query, Finalize() gets called. The Finalize() call should only add extra client wait time if fetch has not completed. Testing: - Added test cases in query_test/test_fetch.py with specific numbers of fetches and verification of the statistics. Change-Id: I9ca525285e03c7b51b04ac292f7b3531e6178218 --- M be/src/service/client-request-state.cc M be/src/service/client-request-state.h M tests/query_test/test_fetch.py 3 files changed, 101 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/97/19897/1 -- To view, visit http://gerrit.cloudera.org:8080/19897 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I9ca525285e03c7b51b04ac292f7b3531e6178218 Gerrit-Change-Number: 19897 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <[email protected]>
