Zoltan Borok-Nagy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11081 )

Change subject: IMPALA-6034: Add scanned bytes limits per query
......................................................................


Patch Set 2: Code-Review+1

(1 comment)

lgtm, thanks for applying the changes!

http://gerrit.cloudera.org:8080/#/c/11081/1/be/src/runtime/coordinator.h
File be/src/runtime/coordinator.h:

http://gerrit.cloudera.org:8080/#/c/11081/1/be/src/runtime/coordinator.h@173
PS1, Line 173: this is the max
             :     /// peak value from any backend.
> I updated the name to reflect that it's a per-host value - the shortened na
Thanks for the explanation!



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3e85f80b70b3fce47e637e9322ed0316ee84f6a9
Gerrit-Change-Number: 11081
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Mostafa Mokhtar <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Tue, 07 Aug 2018 12:50:14 +0000
Gerrit-HasComments: Yes

Reply via email to