Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12261 )

Change subject: [spark] Add write duration histograms
......................................................................


Patch Set 1: Code-Review+1

(1 comment)

LGTM, but I'll defer to Grant since I don't think I fluent in Scala.

http://gerrit.cloudera.org:8080/#/c/12261/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/12261/1//COMMIT_MSG@21
PS1, Line 21: need to be shipped between executors and the driver, so
            : their (serialized) size is relevant
> When tasks end and on executor heartbeat. Executor heartbeat is every 10s (
Thanks for the clarification.



--
To view, visit http://gerrit.cloudera.org:8080/12261
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0fd4d380b08bd7d7d5c1e65b79cffb44a9b9d433
Gerrit-Change-Number: 12261
Gerrit-PatchSet: 1
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Mon, 04 Feb 2019 22:29:17 +0000
Gerrit-HasComments: Yes

Reply via email to