You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2017/03/09 22:06:18 UTC
[kudu-CR] docs: release notes for 1.3
Hello Dan Burkert, David Ribeiro Alves, Jean-Daniel Cryans, Mike Percy, Adar Dembo, Alexey Serbin,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/6334
to review the following change.
Change subject: docs: release notes for 1.3
......................................................................
docs: release notes for 1.3
Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
---
M docs/known_issues.adoc
M docs/prior_release_notes.adoc
M docs/release_notes.adoc
3 files changed, 334 insertions(+), 155 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/34/6334/1
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
[kudu-CR] docs: release notes for 1.3
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Hello Dan Burkert, David Ribeiro Alves, Jean-Daniel Cryans, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6334
to look at the new patch set (#6).
Change subject: docs: release notes for 1.3
......................................................................
docs: release notes for 1.3
Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
---
M docs/known_issues.adoc
M docs/prior_release_notes.adoc
M docs/release_notes.adoc
3 files changed, 342 insertions(+), 156 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/34/6334/6
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] docs: release notes for 1.3
Posted by "Jean-Daniel Cryans (Code Review)" <ge...@cloudera.org>.
Jean-Daniel Cryans has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 5: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No
[kudu-CR] docs: release notes for 1.3
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6334
to look at the new patch set (#3).
Change subject: docs: release notes for 1.3
......................................................................
docs: release notes for 1.3
Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
---
M docs/known_issues.adoc
M docs/prior_release_notes.adoc
M docs/release_notes.adoc
3 files changed, 336 insertions(+), 155 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/34/6334/3
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] docs: release notes for 1.3
Posted by "David Ribeiro Alves (Code Review)" <ge...@cloudera.org>.
David Ribeiro Alves has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 5: Code-Review+2
from my side. leaving it open if other folks want to look
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No
[kudu-CR] docs: release notes for 1.3
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 5:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/6334/5/docs/known_issues.adoc
File docs/known_issues.adoc:
PS5, Line 96: Use of a single Kudu client instance
: for more than one week is not supported.
> Well, it could be possible if running masters with the --authn_token_validi
eh, let's not add too much detail here in release notes. we can always provide that workaround if someone asks, and/or include it in more detailed security documentation.
http://gerrit.cloudera.org:8080/#/c/6334/5/docs/release_notes.adoc
File docs/release_notes.adoc:
PS5, Line 111:
> nit: an extra space
Done
PS5, Line 111: an
> nit: I would expect "the 'ALTER TABLE'" but I might miss something.
Done
PS5, Line 147: without limitations
> nit: this contradicts with the security-related information below. Conside
Done
PS5, Line 189: client
> nit: here and below 'client' --> 'client library' ?
Done
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] docs: release notes for 1.3
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 6: Code-Review+2
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6334/5/docs/known_issues.adoc
File docs/known_issues.adoc:
PS5, Line 96: Use of a single Kudu client instance
: for more than one week is not supported.
> eh, let's not add too much detail here in release notes. we can always prov
This sounds good to me.
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] docs: release notes for 1.3
Posted by "David Ribeiro Alves (Code Review)" <ge...@cloudera.org>.
David Ribeiro Alves has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 1:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/6334/1/docs/known_issues.adoc
File docs/known_issues.adoc:
PS1, Line 83: coarse-grained level
maybe: ... at a system-wide, coarse-grained level....
http://gerrit.cloudera.org:8080/#/c/6334/1/docs/prior_release_notes.adoc
File docs/prior_release_notes.adoc:
PS1, Line 63: SCAN_AT_SNAPSHOT
READ_AT_SNAPSHOT
http://gerrit.cloudera.org:8080/#/c/6334/1/docs/release_notes.adoc
File docs/release_notes.adoc:
PS1, Line 83: +
is this on purpose? pardon my adoc ignorance if so
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] docs: release notes for 1.3
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6334
to look at the new patch set (#2).
Change subject: docs: release notes for 1.3
......................................................................
docs: release notes for 1.3
Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
---
M docs/known_issues.adoc
M docs/prior_release_notes.adoc
M docs/release_notes.adoc
3 files changed, 336 insertions(+), 155 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/34/6334/2
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] docs: release notes for 1.3
Posted by "Dan Burkert (Code Review)" <ge...@cloudera.org>.
Dan Burkert has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 5: Code-Review+1
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No
[kudu-CR] docs: release notes for 1.3
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 1:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/6334/1/docs/prior_release_notes.adoc
File docs/prior_release_notes.adoc:
Line 54: // TODO(danburkert): this flag is marked experimental, should we not doc it?
> Guess we can remove this.
Done
Line 89: // TODO(danburkert): this new flag is marked experimental, should we make it
> And this.
Done
http://gerrit.cloudera.org:8080/#/c/6334/1/docs/release_notes.adoc
File docs/release_notes.adoc:
Line 36: * Kudu 1.3 adds support for strong authentication based on Kerberos. This optional feature
> Just for consistency, you always say "Kudu 1.3" in this section but not alw
Will change it so that in cases where we say "adds", it says "Kudu 1.3 adds", but if we are just saying "Kudu now ..." will not use the versio number.
Line 61:
> Add somewhere a mention that Flume isn't integrated with authn?
adding to known limitations
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] docs: release notes for 1.3
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 1:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/6334/1/docs/known_issues.adoc
File docs/known_issues.adoc:
PS1, Line 83: coarse-grained level
> maybe: ... at a system-wide, coarse-grained level....
Done
http://gerrit.cloudera.org:8080/#/c/6334/1/docs/prior_release_notes.adoc
File docs/prior_release_notes.adoc:
PS1, Line 63: SCAN_AT_SNAPSHOT
> READ_AT_SNAPSHOT
this is from the 1.2 release, so I dont want to edit history (this is just moved from "current" to "prior" release notes)
http://gerrit.cloudera.org:8080/#/c/6334/1/docs/release_notes.adoc
File docs/release_notes.adoc:
PS1, Line 83: +
> is this on purpose? pardon my adoc ignorance if so
yea, it's a weird way in which you continue a bullet-point paragraph.
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] docs: release notes for 1.3
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 6: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No
[kudu-CR] docs: release notes for 1.3
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 5:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/6334/5/docs/known_issues.adoc
File docs/known_issues.adoc:
PS5, Line 96: Use of a single Kudu client instance
: for more than one week is not supported.
Well, it could be possible if running masters with the --authn_token_validity_seconds=<desired_period_of_validity_in_seconds>
Not sure if it's worth mentioning that, though.
http://gerrit.cloudera.org:8080/#/c/6334/5/docs/release_notes.adoc
File docs/release_notes.adoc:
PS5, Line 111: an
nit: I would expect "the 'ALTER TABLE'" but I might miss something.
PS5, Line 111:
nit: an extra space
PS5, Line 147: without limitations
nit: this contradicts with the security-related information below. Consider replacing with something like '..., except for some cases of tightened security restrictions (see below for details).
PS5, Line 189: client
nit: here and below 'client' --> 'client library' ?
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] docs: release notes for 1.3
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has submitted this change and it was merged.
Change subject: docs: release notes for 1.3
......................................................................
docs: release notes for 1.3
Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Reviewed-on: http://gerrit.cloudera.org:8080/6334
Reviewed-by: Alexey Serbin <as...@cloudera.com>
Tested-by: Todd Lipcon <to...@apache.org>
---
M docs/known_issues.adoc
M docs/prior_release_notes.adoc
M docs/release_notes.adoc
3 files changed, 342 insertions(+), 156 deletions(-)
Approvals:
Todd Lipcon: Verified
Alexey Serbin: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 7
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] docs: release notes for 1.3
Posted by "Jean-Daniel Cryans (Code Review)" <ge...@cloudera.org>.
Jean-Daniel Cryans has posted comments on this change.
Change subject: docs: release notes for 1.3
......................................................................
Patch Set 1:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/6334/1/docs/prior_release_notes.adoc
File docs/prior_release_notes.adoc:
Line 54: // TODO(danburkert): this flag is marked experimental, should we not doc it?
Guess we can remove this.
Line 89: // TODO(danburkert): this new flag is marked experimental, should we make it
And this.
http://gerrit.cloudera.org:8080/#/c/6334/1/docs/release_notes.adoc
File docs/release_notes.adoc:
Line 36: * Kudu 1.3 adds support for strong authentication based on Kerberos. This optional feature
Just for consistency, you always say "Kudu 1.3" in this section but not always after.
Line 61:
Add somewhere a mention that Flume isn't integrated with authn?
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] docs: release notes for 1.3
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6334
to look at the new patch set (#5).
Change subject: docs: release notes for 1.3
......................................................................
docs: release notes for 1.3
Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
---
M docs/known_issues.adoc
M docs/prior_release_notes.adoc
M docs/release_notes.adoc
3 files changed, 340 insertions(+), 155 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/34/6334/5
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] docs: release notes for 1.3
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6334
to look at the new patch set (#4).
Change subject: docs: release notes for 1.3
......................................................................
docs: release notes for 1.3
Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
---
M docs/known_issues.adoc
M docs/prior_release_notes.adoc
M docs/release_notes.adoc
3 files changed, 336 insertions(+), 155 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/34/6334/4
--
To view, visit http://gerrit.cloudera.org:8080/6334
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic974d1649859b202ecd2bd5d5eab5ffca3b8bcd8
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>