Rahul Shivu Mahadev has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10560 )

Change subject: Fix Indents from IMPALA-4970
......................................................................


Patch Set 2:

> Thanks for your contribution!
 >
 > One nit about the commit message:
 >
 > https://cwiki.apache.org/confluence/display/IMPALA/Contributing+to+Impala
 >
 > "Each [line of the commit message] should have 72 or fewer
 > characters if possible. The first line should have an empty line
 > after it, and the first line should begin with the ticket(s)
 > addressed, followed by a colon and a space: 'IMPALA-1234: '."

Done, thank you


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4ea53e0780cca9f791808c2a73c7f72ba27c2159
Gerrit-Change-Number: 10560
Gerrit-PatchSet: 2
Gerrit-Owner: Rahul Shivu Mahadev <[email protected]>
Gerrit-Reviewer: Jim Apple <[email protected]>
Gerrit-Reviewer: Rahul Shivu Mahadev <[email protected]>
Gerrit-Reviewer: Sailesh Mukil <[email protected]>
Gerrit-Reviewer: Thomas Marshall <[email protected]>
Gerrit-Reviewer: Vuk Ercegovac <[email protected]>
Gerrit-Comment-Date: Tue, 05 Jun 2018 17:48:51 +0000
Gerrit-HasComments: No

Reply via email to