[kudu-CR] Replace incubator-kudu links with kudu in docs

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

Change subject: Replace incubator-kudu links with kudu in docs
..


Patch Set 4: Code-Review+2

-- 
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: 4
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: No


[kudu-CR] Replace incubator-kudu links with kudu in docs

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

Change subject: Replace incubator-kudu links with kudu in docs
..


Patch Set 4: Code-Review+1

Thanks, this looks good to me!

P.S.  I would give it +2 but I don't have the privilege :)

-- 
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: 4
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: No


[kudu-CR] Replace incubator-kudu links with kudu in docs

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

Change subject: Replace incubator-kudu links with kudu in docs
..


Patch Set 4:

(3 comments)

Alex and Will, thanks for the comments. Let me know if anything else is needed 
here.

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

Line 7: Replace incubator-kudu links with kudu in docs
> Could you re-format the commit message a bit to follow the git guideline:
Done


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

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


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

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


-- 
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: 4
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] Replace incubator-kudu links with kudu in docs

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

Change subject: Replace incubator-kudu links with kudu in docs
..


Patch Set 4:

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

-- 
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: 4
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: No


[kudu-CR] Replace incubator-kudu links with kudu in docs

2016-08-24 Thread Mladen Kovacevic (Code Review)
Hello Will Berkeley, Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

http://gerrit.cloudera.org:8080/4107

to look at the new patch set (#4).

Change subject: Replace incubator-kudu links with kudu in docs
..

Replace incubator-kudu links with kudu in docs

A number of docs were referring to the old incubator link:

  https://github.com/apache/incubator-kudu

As opposed to the new, non-incubator link.

  https://github.com/apache/kudu

We modify several of the documentation files to ensure that the
links are consistent, and up-to-date with the apache/kudu url.

Change-Id: Id9b0560f073735e55243643ddada6eac0001b9d4
---
M RELEASING.adoc
M docs/contributing.adoc
M docs/developing.adoc
M docs/installation.adoc
M docs/release_notes.adoc
5 files changed, 17 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/07/4107/4
-- 
To view, visit http://gerrit.cloudera.org:8080/4107
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id9b0560f073735e55243643ddada6eac0001b9d4
Gerrit-PatchSet: 4
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