[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-02-24 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5829/1//COMMIT_MSG Commit Message: PS1, Line 7: Add .pep8rc

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-02-23 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: I didn't have time to look into this, but I also found yapf, a clang-format based python formatter. It is

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-02-23 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: > Any news with this? I should point out we had a discussion here:

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-02-22 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: Any news with this? -- To view, visit http://gerrit.cloudera.org:8080/5829 To unsubscribe, visit

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-01-31 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5829/1/.pep8rc File .pep8rc: PS1, Line 5: # E251 - Remove

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-01-31 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5829/1/.pep8rc File .pep8rc: PS1, Line 2: # E101 -

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-01-31 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5829/1/.pep8rc File .pep8rc: PS1, Line 5: # E251 - Remove

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-01-31 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: Add .pep8rc for Impala's Python style .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5829/1/.pep8rc File .pep8rc: PS1, Line 2: # E101 -

[Impala-ASF-CR] Add .pep8rc for Impala's Python style

2017-01-30 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/5829 Change subject: Add .pep8rc for Impala's Python style .. Add .pep8rc for Impala's Python style Change-Id: