Jim Apple has posted comments on this change.

Change subject: IMPALA-3327: Disable metric validation for backend clients
......................................................................


Patch Set 3: Code-Review+1

(1 comment)

I'm OK with this if Henry's on-board.

http://gerrit.cloudera.org:8080/#/c/2788/3/tests/verifiers/metric_verifier.py
File tests/verifiers/metric_verifier.py:

Line 21:                # Disable checking of 
impala-server.backends.client-cache.clients-in-use
Can you reword to "TODO: Re-enable ..."?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I67e8c5bd8ab4f71a3c10e9387ae165c0c1dd30f3
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Lars Volker <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Jim Apple <[email protected]>
Gerrit-HasComments: Yes

Reply via email to