Henry Robinson has posted comments on this change.

Change subject: Match .clang-format more closely to actual practice.
......................................................................


Patch Set 2:

> I'm not sure I understand why a person might want to run the inferencer 
> again, even if the relationship between config files and the induced changes 
> are sometimes confusing.

If I don't understand how to get the results I want (because the formatter has 
become a black box), the only recourse left to me is to ask a machine to do it 
for me - just like has happened here.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iccaec6c1673c3e08d2c39200b0c84437af629aed
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Jim Apple <jbap...@cloudera.com>
Gerrit-Reviewer: Henry Robinson <he...@cloudera.com>
Gerrit-Reviewer: Jim Apple <jbap...@cloudera.com>
Gerrit-Reviewer: Marcel Kornacker <mar...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-HasComments: No

Reply via email to