Mike Percy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11896 )

Change subject: docs: Use a non-SSH method of installing Gerrit commit hook
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/11896/1/docs/contributing.adoc
File docs/contributing.adoc:

http://gerrit.cloudera.org:8080/#/c/11896/1/docs/contributing.adoc@72
PS1, Line 72: <username>@
> I think it's possible to drop username, no?
Great point!


http://gerrit.cloudera.org:8080/#/c/11896/1/docs/contributing.adoc@72
PS1, Line 72: -L
> I think it's a bit more robust and less chatty with '-s -S' options (that m
ah yes, -LSsf actually looks pretty good



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibdb024aafcd601f740ae73d83c54f9659cc5a3a9
Gerrit-Change-Number: 11896
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Comment-Date: Wed, 07 Nov 2018 04:23:53 +0000
Gerrit-HasComments: Yes

Reply via email to