You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Grant Henke (Code Review)" <ge...@cloudera.org> on 2018/05/02 18:46:41 UTC
[kudu-CR](branch-1.7.x) Release notes for 1.7.1
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10282
Change subject: Release notes for 1.7.1
......................................................................
Release notes for 1.7.1
Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
---
M docs/release_notes.adoc
1 file changed, 43 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/82/10282/1
--
To view, visit http://gerrit.cloudera.org:8080/10282
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.7.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
Gerrit-Change-Number: 10282
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
[kudu-CR](branch-1.7.x) Release notes for 1.7.1
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10282 )
Change subject: Release notes for 1.7.1
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/10282
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.7.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
Gerrit-Change-Number: 10282
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 02 May 2018 19:53:00 +0000
Gerrit-HasComments: No
[kudu-CR](branch-1.7.x) Release notes for 1.7.1
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10282 )
Change subject: Release notes for 1.7.1
......................................................................
Patch Set 1:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/10282/1/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/10282/1/docs/release_notes.adoc@31
PS1, Line 31: 0
1 ? Or this should be read as 'issues fixed in 1.7.0'?
http://gerrit.cloudera.org:8080/#/c/10282/1/docs/release_notes.adoc@38
PS1, Line 38:
nit: extra space
http://gerrit.cloudera.org:8080/#/c/10282/1/docs/release_notes.adoc@49
PS1, Line 49: Added an extra check in `ksck` to verify the tablet server id matches
: the expected id. Without this `ksck` could report a successful connection
: to extra servers if one was removed and replaced on the same host.
: (see link:https://issues.apache.org/jira/browse/KUDU-2364[KUDU-2364])
I would clarify on this with Will and Andrew, since Andrew found https://issues.apache.org/jira/browse/KUDU-2426
Maybe, it would be necessary to integrate a fix for KUDU-2426 as well since fix for KUDU-2364 could introduce KUDU-2426.
http://gerrit.cloudera.org:8080/#/c/10282/1/docs/release_notes.adoc@68
PS1, Line 68: to
too
--
To view, visit http://gerrit.cloudera.org:8080/10282
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.7.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
Gerrit-Change-Number: 10282
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 02 May 2018 19:03:37 +0000
Gerrit-HasComments: Yes
[kudu-CR](branch-1.7.x) Release notes for 1.7.1
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10282 )
Change subject: Release notes for 1.7.1
......................................................................
Patch Set 1:
The website has version based directories for this sort of thing. I modeled this commit after the 1.3.1 release notes:
http://kudu.apache.org/releases/1.3.1/docs/release_notes.html
--
To view, visit http://gerrit.cloudera.org:8080/10282
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.7.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
Gerrit-Change-Number: 10282
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 02 May 2018 19:14:44 +0000
Gerrit-HasComments: No
[kudu-CR](branch-1.7.x) Release notes for 1.7.1
Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10282 )
Change subject: Release notes for 1.7.1
......................................................................
Patch Set 1: Code-Review+2
Is this something we typically do for point releases? The website isn't branched, so doesn't this belong in 'master'?
--
To view, visit http://gerrit.cloudera.org:8080/10282
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.7.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
Gerrit-Change-Number: 10282
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Comment-Date: Wed, 02 May 2018 18:52:56 +0000
Gerrit-HasComments: No
[kudu-CR](branch-1.7.x) Release notes for 1.7.1
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10282 )
Change subject: Release notes for 1.7.1
......................................................................
Patch Set 1:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/10282/1/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/10282/1/docs/release_notes.adoc@31
PS1, Line 31: 0
> 1 ? Or this should be read as 'issues fixed in 1.7.0'?
Done
http://gerrit.cloudera.org:8080/#/c/10282/1/docs/release_notes.adoc@38
PS1, Line 38:
> nit: extra space
Done
http://gerrit.cloudera.org:8080/#/c/10282/1/docs/release_notes.adoc@49
PS1, Line 49: Added an extra check in `ksck` to verify the tablet server id matches
: the expected id. Without this `ksck` could report a successful connection
: to extra servers if one was removed and replaced on the same host.
: (see link:https://issues.apache.org/jira/browse/KUDU-2364[KUDU-2364])
> I would clarify on this with Will and Andrew, since Andrew found https://is
Thanks for the heads up. I saw you commented on the jira. I will follow along there.
http://gerrit.cloudera.org:8080/#/c/10282/1/docs/release_notes.adoc@68
PS1, Line 68: to
> too
Done
--
To view, visit http://gerrit.cloudera.org:8080/10282
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.7.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
Gerrit-Change-Number: 10282
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 02 May 2018 19:23:39 +0000
Gerrit-HasComments: Yes
[kudu-CR](branch-1.7.x) Release notes for 1.7.1
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10282
to look at the new patch set (#2).
Change subject: Release notes for 1.7.1
......................................................................
Release notes for 1.7.1
Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
---
M docs/release_notes.adoc
1 file changed, 43 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/82/10282/2
--
To view, visit http://gerrit.cloudera.org:8080/10282
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.7.x
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
Gerrit-Change-Number: 10282
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
[kudu-CR](branch-1.7.x) Release notes for 1.7.1
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10282 )
Change subject: Release notes for 1.7.1
......................................................................
Release notes for 1.7.1
Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
Reviewed-on: http://gerrit.cloudera.org:8080/10282
Reviewed-by: Alexey Serbin <as...@cloudera.com>
Tested-by: Kudu Jenkins
---
M docs/release_notes.adoc
1 file changed, 38 insertions(+), 1 deletion(-)
Approvals:
Alexey Serbin: Looks good to me, approved
Kudu Jenkins: Verified
--
To view, visit http://gerrit.cloudera.org:8080/10282
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.7.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
Gerrit-Change-Number: 10282
Gerrit-PatchSet: 4
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
[kudu-CR](branch-1.7.x) Release notes for 1.7.1
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10282
to look at the new patch set (#3).
Change subject: Release notes for 1.7.1
......................................................................
Release notes for 1.7.1
Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
---
M docs/release_notes.adoc
1 file changed, 38 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/82/10282/3
--
To view, visit http://gerrit.cloudera.org:8080/10282
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.7.x
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4415544947904665f7ccd9698468d1b28af70d0d
Gerrit-Change-Number: 10282
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>