Lars Volker has posted comments on this change.

Change subject: Add .clang-format for Impala's C++ style
......................................................................


Patch Set 1:

Thank you for making this file. I think it would be extremely helpful to agree 
on some machine-checkable style and align all of our code with it. Maybe now is 
a good time for such a change, given we recently had a couple of large, 
style-related changes ("> >" to ">>", boost to std, licenses). LMK if I can 
help driving this.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I274c5993c7be344fc4b7729d21a13da993f9f3aa
Gerrit-PatchSet: 1
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: Lars Volker <l...@cloudera.com>
Gerrit-HasComments: No

Reply via email to