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 2019/06/17 16:34:28 UTC
[kudu-CR] [docs] Add 1.10.0 release notes
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13662
Change subject: [docs] Add 1.10.0 release notes
......................................................................
[docs] Add 1.10.0 release notes
Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
---
M docs/release_notes.adoc
1 file changed, 207 insertions(+), 10 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/62/13662/1
--
To view, visit http://gerrit.cloudera.org:8080/13662
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
Gerrit-Change-Number: 13662
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
[kudu-CR] [docs] Add 1.10.0 release notes
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Hello Will Berkeley, Mike Percy, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/13662
to look at the new patch set (#2).
Change subject: [docs] Add 1.10.0 release notes
......................................................................
[docs] Add 1.10.0 release notes
Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
---
M docs/release_notes.adoc
1 file changed, 207 insertions(+), 10 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/62/13662/2
--
To view, visit http://gerrit.cloudera.org:8080/13662
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
Gerrit-Change-Number: 13662
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: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
[kudu-CR] [docs] Add 1.10.0 release notes
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13662 )
Change subject: [docs] Add 1.10.0 release notes
......................................................................
Patch Set 1: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/13662
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
Gerrit-Change-Number: 13662
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: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Mon, 17 Jun 2019 17:25:42 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] Add 1.10.0 release notes
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has abandoned this change. ( http://gerrit.cloudera.org:8080/13662 )
Change subject: [docs] Add 1.10.0 release notes
......................................................................
Abandoned
See https://gerrit.cloudera.org/#/c/13669/
--
To view, visit http://gerrit.cloudera.org:8080/13662
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: abandon
Gerrit-Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
Gerrit-Change-Number: 13662
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: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
[kudu-CR] [docs] Add 1.10.0 release notes
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13662 )
Change subject: [docs] Add 1.10.0 release notes
......................................................................
Patch Set 1:
(9 comments)
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@49
PS1, Line 49: Spark backup
I think this phrasing is a little confusing. Perhaps a backup job implemented using Apache Spark? Or "a job which can run on an Apache Spark cluster"? or "using Apache Spark as an execution environment" or something? As it's written it sounds like a job which backs up Spark
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@55
PS1, Line 55: HMS synchronization documentation
: for more details.
can this be a link?
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@61
PS1, Line 61: authorization documentation
link?
Worth noting that this only works when HMS integration is enabled (right? or can it be enabled independently?)
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@69
PS1, Line 69: though this
: may be overridden using `--max_column_comment_length`
Do we want to document this? We didn't tag this flag as stable (though I guess we also didn't call it unsafe or experimental).
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@83
PS1, Line 83: helm
Helm?
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@83
PS1, Line 83: statefulset
I think this is camel-cased StatefulSet
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@96
PS1, Line 96: * `kudu table copy` copies data from one table to another, within the same
: cluster or across clusters.
Do we want to say something here to indicate this is driven by a single client and therefore not suitable for large tables?
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@113
PS1, Line 113: * The performance of mutations (i.e. UPDATE, DELETE, and re-INSERT) to
can this be combined with the "Update performance" one above?
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@137
PS1, Line 137: Once resolved DNS records
I think it's simpler phrasing to write "DNS records are now cached" or "DNS resolutions are now cached"
--
To view, visit http://gerrit.cloudera.org:8080/13662
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
Gerrit-Change-Number: 13662
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: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Mon, 17 Jun 2019 18:16:56 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Add 1.10.0 release notes
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13662 )
Change subject: [docs] Add 1.10.0 release notes
......................................................................
Patch Set 1:
(9 comments)
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@49
PS1, Line 49: Spark backup
> I think this phrasing is a little confusing. Perhaps a backup job implement
Done
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@55
PS1, Line 55: HMS synchronization documentation
: for more details.
> can this be a link?
The link will be added when the docs are added.
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@61
PS1, Line 61: authorization documentation
The link will be added when the docs are added.
> Worth noting that this only works when HMS integration is enabled (right? or can it be enabled independently?)
I think this will be in the docs.
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@69
PS1, Line 69: though this
: may be overridden using `--max_column_comment_length`
> Do we want to document this? We didn't tag this flag as stable (though I gu
Happy to remove it for simplicity.
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@83
PS1, Line 83: helm
> Helm?
Done
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@83
PS1, Line 83: statefulset
> I think this is camel-cased StatefulSet
Done
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@96
PS1, Line 96: * `kudu table copy` copies data from one table to another, within the same
: cluster or across clusters.
> Do we want to say something here to indicate this is driven by a single cli
I can add a sentence indicating that.
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@113
PS1, Line 113: * The performance of mutations (i.e. UPDATE, DELETE, and re-INSERT) to
> can this be combined with the "Update performance" one above?
Done
http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@137
PS1, Line 137: Once resolved DNS records
> I think it's simpler phrasing to write "DNS records are now cached" or "DNS
Done
--
To view, visit http://gerrit.cloudera.org:8080/13662
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
Gerrit-Change-Number: 13662
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: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Mon, 17 Jun 2019 18:31:35 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Add 1.10.0 release notes
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13662 )
Change subject: [docs] Add 1.10.0 release notes
......................................................................
Patch Set 1:
A rendered version of the doc can be seen here: https://github.com/granthenke/kudu/blob/release-notes/docs/release_notes.adoc
--
To view, visit http://gerrit.cloudera.org:8080/13662
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
Gerrit-Change-Number: 13662
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: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Mon, 17 Jun 2019 16:36:12 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] Add 1.10.0 release notes
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has removed a vote on this change.
Change subject: [docs] Add 1.10.0 release notes
......................................................................
Removed Verified-1 by Kudu Jenkins (120)
--
To view, visit http://gerrit.cloudera.org:8080/13662
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
Gerrit-Change-Number: 13662
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: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>