Vuk Ercegovac has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11191 )

Change subject: IMPALA-7408: add a flag to disable reading fs data from catalogd
......................................................................


Patch Set 5:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/11191/5//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11191/5//COMMIT_MSG@7
PS5, Line 7:
> nit: ..debugging...
Done


http://gerrit.cloudera.org:8080/#/c/11191/5/be/src/common/global-flags.cc
File be/src/common/global-flags.cc:

http://gerrit.cloudera.org:8080/#/c/11191/5/be/src/common/global-flags.cc@204
PS5, Line 204: Examples when
> ?
re-worded.


http://gerrit.cloudera.org:8080/#/c/11191/5/tests/custom_cluster/test_disable_catalog_data_ops.py
File tests/custom_cluster/test_disable_catalog_data_ops.py:

http://gerrit.cloudera.org:8080/#/c/11191/5/tests/custom_cluster/test_disable_catalog_data_ops.py@42
PS5, Line 42: /
> flake8: E501 line too long (106 > 90 characters)
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I15789fb489b285e2a6565025eb17c63cdc726354
Gerrit-Change-Number: 11191
Gerrit-PatchSet: 5
Gerrit-Owner: Vuk Ercegovac <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Vuk Ercegovac <[email protected]>
Gerrit-Comment-Date: Tue, 14 Aug 2018 17:03:58 +0000
Gerrit-HasComments: Yes

Reply via email to