[kudu-CR] Update contributing doc page with apache/kudu instead of apache/incubator-kudu

2016-08-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change.

Change subject: Update contributing doc page with apache/kudu instead of 
apache/incubator-kudu
..


Patch Set 3:

(1 comment)

Could you also update the RELEASING.adoc accordingly?

http://gerrit.cloudera.org:8080/#/c/4107/3//COMMIT_MSG
Commit Message:

Line 7: Update contributing doc page with apache/kudu instead
Could you re-format the commit message a bit to follow the git guideline:

https://git-scm.com/book/en/v2/Distributed-Git-Contributing-to-a-Project#_commit_guidelines


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id9b0560f073735e55243643ddada6eac0001b9d4
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mladen Kovacevic 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mladen Kovacevic 
Gerrit-Reviewer: Will Berkeley 
Gerrit-HasComments: Yes


[kudu-CR] Update contributing doc page with apache/kudu instead of apache/incubator-kudu

2016-08-24 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change.

Change subject: Update contributing doc page with apache/kudu instead of 
apache/incubator-kudu
..


Patch Set 3:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/4107/3/docs/contributing.adoc
File docs/contributing.adoc:

Line 52: git clone https://github.com/apache/kudu.git kudu
now that the repo name is kudu, I think this can be simplified to

git clone https://github.com/apache/kudu


http://gerrit.cloudera.org:8080/#/c/4107/3/docs/installation.adoc
File docs/installation.adoc:

Line 242: $ git clone https://github.com/apache/kudu kudu
same with all these; the kudu at the end can be left off.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id9b0560f073735e55243643ddada6eac0001b9d4
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mladen Kovacevic 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mladen Kovacevic 
Gerrit-Reviewer: Will Berkeley 
Gerrit-HasComments: Yes


[kudu-CR] Update contributing doc page with apache/kudu instead of apache/incubator-kudu

2016-08-23 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change.

Change subject: Update contributing doc page with apache/kudu instead of 
apache/incubator-kudu
..


Patch Set 3: Code-Review+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id9b0560f073735e55243643ddada6eac0001b9d4
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mladen Kovacevic 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mladen Kovacevic 
Gerrit-Reviewer: Will Berkeley 
Gerrit-HasComments: No


[kudu-CR] Update contributing doc page with apache/kudu instead of apache/incubator-kudu

2016-08-23 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has posted comments on this change.

Change subject: Update contributing doc page with apache/kudu instead of 
apache/incubator-kudu
..


Patch Set 3:

> There's a bunch of similar git clone links that use incubator-kudu
 > instead of kudu in installation.adoc. Could you fix those too?
 > Thanks!

Sure, did a quick grep, and found some changes in developing, installation and 
release_notes.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id9b0560f073735e55243643ddada6eac0001b9d4
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mladen Kovacevic 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mladen Kovacevic 
Gerrit-Reviewer: Will Berkeley 
Gerrit-HasComments: No


[kudu-CR] Update contributing doc page with apache/kudu instead of apache/incubator-kudu

2016-08-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change.

Change subject: Update contributing doc page with apache/kudu instead of 
apache/incubator-kudu
..


Patch Set 3:

Build Started http://104.196.14.100/job/kudu-gerrit/3045/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id9b0560f073735e55243643ddada6eac0001b9d4
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mladen Kovacevic 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Will Berkeley 
Gerrit-HasComments: No