[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-15 Thread Mike Percy (Code Review) 
Mike Percy has submitted this change and it was merged.

Change subject: KUDU-1487 Add instructions for cutting a release
..


KUDU-1487 Add instructions for cutting a release

Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Reviewed-on: http://gerrit.cloudera.org:8080/3614
Tested-by: Kudu Jenkins
Reviewed-by: Mike Percy 
---
A RELEASING.adoc
1 file changed, 195 insertions(+), 0 deletions(-)

Approvals:
  Mike Percy: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 7
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Reviewer: Misty Stanley-Jones

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-14 Thread Jean-Daniel Cryans (Code Review) 
Hello Kudu Jenkins,

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

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

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

Change subject: KUDU-1487 Add instructions for cutting a release
..

KUDU-1487 Add instructions for cutting a release

Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
---
A RELEASING.adoc
1 file changed, 195 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/14/3614/6
-- 
To view, visit http://gerrit.cloudera.org:8080/3614
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones

[kudu-CR] KUDU-1487 Add instructions for cutting a release

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

Change subject: KUDU-1487 Add instructions for cutting a release
..


Patch Set 6:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones 
Gerrit-HasComments: No

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-13 Thread Misty Stanley-Jones (Code Review) 
Hello Kudu Jenkins,

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

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

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

Change subject: KUDU-1487 Add instructions for cutting a release
..

KUDU-1487 Add instructions for cutting a release

Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
---
A RELEASING.adoc
1 file changed, 196 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/14/3614/5
-- 
To view, visit http://gerrit.cloudera.org:8080/3614
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-13 Thread Jean-Daniel Cryans (Code Review) 
Jean-Daniel Cryans has posted comments on this change.

Change subject: KUDU-1487 Add instructions for cutting a release
..


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3614/4/RELEASING.adoc
File RELEASING.adoc:

Line 70: . Check out the `master` branch. Bump the version in `version.txt` in 
the `master`
This section doesn't look right once rendered.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones 
Gerrit-HasComments: Yes

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-13 Thread Misty Stanley-Jones (Code Review) 
Hello Kudu Jenkins,

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

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

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

Change subject: KUDU-1487 Add instructions for cutting a release
..

KUDU-1487 Add instructions for cutting a release

Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
---
A RELEASING.adoc
1 file changed, 195 insertions(+), 0 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-13 Thread Jean-Daniel Cryans (Code Review) 
Jean-Daniel Cryans has posted comments on this change.

Change subject: KUDU-1487 Add instructions for cutting a release
..


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3614/3/RELEASING.adoc
File RELEASING.adoc:

PS3, Line 70: Bump the version `version.txt` in the `master`
:   branch using the following command
It's two things:
 - You need to bump the version in version.txt
 - You need to run "mvn versions:set -DnewVersion=0.X.0-SNAPSHOT" in java/.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones 
Gerrit-HasComments: Yes

[kudu-CR] KUDU-1487 Add instructions for cutting a release

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

Change subject: KUDU-1487 Add instructions for cutting a release
..


Patch Set 3:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones 
Gerrit-HasComments: No

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-13 Thread Jean-Daniel Cryans (Code Review) 
Jean-Daniel Cryans has posted comments on this change.

Change subject: KUDU-1487 Add instructions for cutting a release
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3614/1/RELEASING.adoc
File RELEASING.adoc:

Line 55: +
> Look for a line that has the mvn versions:set thing, right? That is what I 
You won't find that anywhere, you need to run something like it.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones 
Gerrit-HasComments: Yes

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-13 Thread Misty Stanley-Jones (Code Review) 
Misty Stanley-Jones has posted comments on this change.

Change subject: KUDU-1487 Add instructions for cutting a release
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3614/1/RELEASING.adoc
File RELEASING.adoc:

Line 55: +
> But it says to look for a line like the following, best I can tell the next
Look for a line that has the mvn versions:set thing, right? That is what I 
understood your draft to mean.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones 
Gerrit-HasComments: Yes

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-13 Thread Jean-Daniel Cryans (Code Review) 
Jean-Daniel Cryans has posted comments on this change.

Change subject: KUDU-1487 Add instructions for cutting a release
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3614/1/RELEASING.adoc
File RELEASING.adoc:

Line 55: +
> Nope, this just makes the codeblock be part of the list item.
But it says to look for a line like the following, best I can tell the next 
line is a command. Or should this be: "Bump the version `version.txt` in the 
`master` branch and use the following command to change the Java version from 
the java/ folder".


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones 
Gerrit-HasComments: Yes

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-13 Thread Misty Stanley-Jones (Code Review) 
Misty Stanley-Jones has posted comments on this change.

Change subject: KUDU-1487 Add instructions for cutting a release
..


Patch Set 1:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/3614/1/RELEASING.adoc
File RELEASING.adoc:

Line 1: = RELEASING KUDU
> I think we need a license header here.
Done


Line 55: +
> Missing a line here?
Nope, this just makes the codeblock be part of the list item.


PS1, Line 90: #might not be necessary?
> Yeah this might not be since we rely on that repo only for gerrit, and we d
Done


Line 149: . Push the tag to the ASF’s git.
> I guess we should add a command line example here, since we do it almost ev
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones 
Gerrit-HasComments: Yes

[kudu-CR] KUDU-1487 Add instructions for cutting a release

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

Change subject: KUDU-1487 Add instructions for cutting a release
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones 
Gerrit-HasComments: No

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-12 Thread Jean-Daniel Cryans (Code Review) 
Jean-Daniel Cryans has posted comments on this change.

Change subject: KUDU-1487 Add instructions for cutting a release
..


Patch Set 1:

(4 comments)

Thanks a lot Misty for taking my original document to the finish line.

http://gerrit.cloudera.org:8080/#/c/3614/1/RELEASING.adoc
File RELEASING.adoc:

Line 1: = RELEASING KUDU
I think we need a license header here.


Line 55: +
Missing a line here?


PS1, Line 90: #might not be necessary?
Yeah this might not be since we rely on that repo only for gerrit, and we don't 
push patches to tags.


Line 149: . Push the tag to the ASF’s git.
I guess we should add a command line example here, since we do it almost 
everywhere else.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

[kudu-CR] KUDU-1487 Add instructions for cutting a release

2016-07-11 Thread Misty Stanley-Jones (Code Review) 
Misty Stanley-Jones has uploaded a new change for review.

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

Change subject: KUDU-1487 Add instructions for cutting a release
..

KUDU-1487 Add instructions for cutting a release

Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
---
A RELEASING.adoc
1 file changed, 176 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/14/3614/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3614
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5b52edb68d35d07ee50bb3c373bf866560f5bc93
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones