You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2019/10/09 18:40:12 UTC
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14404
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
WIP [docs] Kudu 1.11 release notes draft
WIP:
* this is published just as working draft to iterate upon
* the branch is wrong: DO NOT COMMIT
Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
---
M docs/release_notes.adoc
1 file changed, 161 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/14404/1
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello honeyhexin, Attila Bukor, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, Grant Henke, Greg Solovyev, Hao Hao, Volodymyr Verovkin, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14404
to look at the new patch set (#4).
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
WIP [docs] Kudu 1.11 release notes draft
The rendered version is available at:
https://gist.github.com/alexeyserbin/53200325d07f4c922682580489e3d45f
WIP:
* this is published just as working draft to iterate upon
* the branch is wrong: DO NOT COMMIT
Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
---
M docs/release_notes.adoc
1 file changed, 193 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/14404/4
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello honeyhexin, Attila Bukor, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, Grant Henke, Greg Solovyev, Hao Hao, Volodymyr Verovkin, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14404
to look at the new patch set (#5).
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
WIP [docs] Kudu 1.11 release notes draft
The rendered version is available at:
https://gist.github.com/alexeyserbin/53200325d07f4c922682580489e3d45f
WIP:
* this is published just as working draft to iterate upon
* the branch is wrong: DO NOT COMMIT
Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
---
M docs/release_notes.adoc
1 file changed, 187 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/14404/5
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Greg Solovyev (Code Review)" <ge...@cloudera.org>.
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/14404 )
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
Patch Set 6:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/14404/6/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/6/docs/release_notes.adoc@157
PS6, Line 157: * A quick-start example of using Apache Spark to load, query, and modify
nit: I'd change these two bullet points to active voice
http://gerrit.cloudera.org:8080/#/c/14404/6/docs/release_notes.adoc@188
PS6, Line 188: * Decoding of deltas' timestamps has been optimized
"Optimized decoding of deltas' timestamps" - I'd use active voice instead of passive, because it is shorter and because we use it in most other items.
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 6
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
Gerrit-Comment-Date: Thu, 10 Oct 2019 22:42:04 +0000
Gerrit-HasComments: Yes
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14404 )
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
Patch Set 3:
(28 comments)
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@51
PS1, Line 51:
> I think it was deprecated in 1.10
Why? d8aeab7f97a76275e6f7f647099c5b339961c623 is not in 1.10
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@51
PS1, Line 51: with clients of earlier ver
> nit: wrap with ``s so it shows up as monospaced code font.
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@63
PS1, Line 63: tablet server maintenance, and the `kudu tserver list` tool is amended with
: a "state" column option to display current state of each tablet server
> Kudu now supports putting tablet servers into maintenance. While a tablet s
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@66
PS1, Line 66:
> nit: a built-in
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@66
PS1, Line 66:
> Might be worth trying to describe what the user should expect if switching
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@73
PS1, Line 73: roject:
> This might be confusing since it might make it seem like the flag is tagged
Removed 'run-time'
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@74
PS1, Line 74: ol.ntp.org`. To switch Kudu
: masters and tablet servers to use the built-in NTP client, add
: `--time_source=builtin` into the set of their flags, customizing the
: `--builtin_ntp_servers` flag as well i
> nit: This seems like a lot of detail for a release note. Maybe we should ad
Removed the details.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@82
PS1, Line 82: now
> nit: "will" so it's obvious that this isn't optional. Or "can now optionall
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@83
PS1, Line 83: ar
> are
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@97
PS1, Line 97:
> nit: `kudu table` tool? Or say that new tools `kudu table get_extra_configs
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@111
PS1, Line 111:
> nit: CFileReaders (plural), and maybe we wrap these in quotes. I.e. as "CFi
Done.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@113
PS1, Line 113: * The `kudu fs dump uuid` CLI tool is now significantly faster and consumes
: significantly less IO.
:
: * The memory consumed by CFileReaders and BloomFileReaders is factored o
> Not sure this deserves a release note unless there are user-facing ways to
Yep, that's fair. Removed.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@123
PS1, Line 123:
: * Tablet server Web UI now supports sorting the list of tables by the columns
: of "Table Name", "Create Time", and "Last Alter Time".
> We should note that this has been enabled by default and maybe drop the men
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@127
PS1, Line 127: lient_locat
> nit: "will now" so it's obvious this is new non-optional behavior
I prefer present tense.
Replaced with 'are now able ...'
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@133
PS1, Line 133: d tablet's data directory group
> nit: "in case no replica is local"
I described why it's beneficial.
I think it would be too much details to describe the previous strange behavior, especially given the fact it's no longer relevant. But I mentioned what it might looked like in prior version.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@135
PS1, Line 135: tablet's group are full
: (see link:https://issues.apache.org/jira/browse/KUDU-2907[KUDU-2907]).
:
> Maybe, "The serialization of in-memory rows to Kudu's wire format has been
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@144
PS1, Line 144: /issues.apa
> nit: capitalize these
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@159
PS1, Line 159: for ta
> nit: previous? Unless this behavior can still be achieved somehow
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@163
PS1, Line 163: a real data set has been added.
> Does this happen by default? If so, use "will now" instead of "can", and if
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@164
PS1, Line 164:
> I don't think this is the right link
It's link:https://issues.apache.org/jira/browse/KUDU-2823 , indeed.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@181
PS1, Line 181: `AlterTableOptions.addRangePartition()` of {cpp} and Java Kudu client API
: correspondingly
: (see link:https://issues.apache.org/jira/browse/KUDU-2823[KUDU-2823]).
> Should this be classified as a bug fix?
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@185
PS1, Line 185: ive for all outbound c
> nit: drop
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@188
PS1, Line 188:
> nit: drop
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@189
PS1, Line 189: The `kudu table scan` and `kudu table copy` CL
> nit: "and NULL/NOT NULL predicates to leverage SIMD operations"
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@196
PS1, Line 196: Optimized the initialization of DeltaMemStore for the case when no matching
: deltas are present
: (see link:https://issues.apache.org/jira/browse/KUDU-2381[KUDU-2381]).
:
: * Scan tokens are now rehydrated so a scan token created before renaming
: a column can be used even after the
> nit: maybe cut down some of the specifics; also not specific to tablet serv
Yep, there is nothing mentioned about tablet server specifics, right.
I changed it to
Fixed an issue with frequent leader re-elections when persisting a Raft transaction into WAL took longer than the leader election timeout. The issue was contributing to the frequent leader change phenomenon (a.k.a. election storms) when tablet servers were under heavy load
Is it any better? I'd like to keep the information on how slow WAL operation should be to trigger the issue since I think that's crucial.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@204
PS1, Line 204: /`NOT NULL` predicates to leverage SIMD instructions
> nit: maybe just:
I changed it to:
Fixed a tablet server crash in case if blocks were not removed due to IO error. This
issue may have surfaced after recovering from a disk failure
Is it any better?
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@207
PS1, Line 207: rn_1.11.0_fixed
> nit: Fixed a crash in
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@272
PS1, Line 272: vice-versa.
> Should note the first-time contributors. I think that's:
Thanks. There is one more, I think: honeyhexin
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
Gerrit-Comment-Date: Wed, 09 Oct 2019 22:59:30 +0000
Gerrit-HasComments: Yes
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello honeyhexin, Attila Bukor, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, Grant Henke, Greg Solovyev, Hao Hao, Volodymyr Verovkin, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14404
to look at the new patch set (#7).
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
WIP [docs] Kudu 1.11 release notes draft
The rendered version is available at:
https://gist.github.com/alexeyserbin/53200325d07f4c922682580489e3d45f
WIP:
* this is published just as working draft to iterate upon
* the branch is wrong (should be in branch-1.11.x): DO NOT COMMIT
into the master branch
Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
---
M docs/release_notes.adoc
1 file changed, 186 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/14404/7
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 7
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14404 )
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
Patch Set 3:
(16 comments)
http://gerrit.cloudera.org:8080/#/c/14404/3//COMMIT_MSG
Commit Message:
http://gerrit.cloudera.org:8080/#/c/14404/3//COMMIT_MSG@7
PS3, Line 7: WIP [docs] Kudu 1.11 release notes draft
Could you also post to your own github branch and post a link so reviewers can read a rendered version?
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@51
PS1, Line 51:
> Why? d8aeab7f97a76275e6f7f647099c5b339961c623 is not in 1.10
Ah you're right. I think the "interned replica" behavior itself was implemented in 1.10, not the deprecation.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@123
PS1, Line 123:
: * Tablet server Web UI now supports sorting the list of tables by the columns
: of "Table Name", "Create Time", and "Last Alter Time".
> Done
Actually upon looking at the commits, I thought the default behavior has changed, but only the tagging has changed.
Omitting the second sentence should be fine, I think. Or maybe elaborate on why a user might want to set this to false. It's not clear upon reading this why this is an improvement.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@127
PS1, Line 127: lient_locat
> I prefer present tense.
My concern isn't about tense. "are now able" and "are able" don't really make clear that the behavior always happens.
How about: "Maybe Tablet servers now expand a tablet's data directory groups with available healthy directories when all directories of that tablet's group are full."
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@133
PS1, Line 133: d tablet's data directory group
> I described why it's beneficial.
Thanks! Yeah, while the previous behavior isn't gone now, I might have found myself wondering why this was an improvement, because I might have thought we would be taking into account location awareness somehow.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@196
PS1, Line 196: Optimized the initialization of DeltaMemStore for the case when no matching
: deltas are present
: (see link:https://issues.apache.org/jira/browse/KUDU-2381[KUDU-2381]).
:
: * Scan tokens are now rehydrated so a scan token created before renaming
: a column can be used even after the
> Yep, there is nothing mentioned about tablet server specifics, right.
Yep, much better. Some tweaks to limit the redundancy (you mention frequent leader changes a couple times), how does this sound:
Fixed an issue that would cause frequent leader elections when Raft transactions to the WAL took longer than the leader election timeout to persist.
Though why was heavy load to tablet servers important? And isn't it not specific to tablet servers? Or was it that heavy load could contribute to a slow WAL?
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@204
PS1, Line 204: /`NOT NULL` predicates to leverage SIMD instructions
> I changed it to:
SGTM
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@272
PS1, Line 272: vice-versa.
> Thanks. There is one more, I think: honeyhexin
Ah, you're right. For some reason, one of my commits from a while ago shows up when I search her name (seems she was a reviewer): https://gerrit.cloudera.org/c/11614/
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@100
PS3, Line 100: m
nit: the, same below
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@140
PS3, Line 140: initiated
nit: "initiating", or "that initiated"
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@179
PS3, Line 179: with specified dimension
nit: "with a dimension"? "with a specific dimension"?
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@212
PS3, Line 212: was contributing to the frequent leader change phenomenon
: (a.k.a. election storms)
nit: could lead to frequent elections.
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@216
PS3, Line 216: case if b
nit: "in cases where"
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@232
PS3, Line 232: Kudu
nit: the Kudu
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@240
PS3, Line 240: pending AlterSchema
: transaction.
nit: plurality is off, "pending `AlterSchema` transactions"? "a pending `AlterSchema` transaction?
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@296
PS3, Line 296: * honeyhexin
On Slack, she goes by Xin He.
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
Gerrit-Comment-Date: Wed, 09 Oct 2019 23:37:14 +0000
Gerrit-HasComments: Yes
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14404 )
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
Patch Set 3:
(14 comments)
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@51
PS1, Line 51:
> Ah you're right. I think the "interned replica" behavior itself was impleme
Ack
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@127
PS1, Line 127: lient_locat
> My concern isn't about tense. "are now able" and "are able" don't really ma
Done
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@133
PS1, Line 133: d tablet's data directory group
> Thanks! Yeah, while the previous behavior isn't gone now, I might have foun
Ack
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@196
PS1, Line 196: Optimized the initialization of DeltaMemStore for the case when no matching
: deltas are present
: (see link:https://issues.apache.org/jira/browse/KUDU-2381[KUDU-2381]).
:
: * Scan tokens are now rehydrated so a scan token created before renaming
: a column can be used even after the
> Yep, much better. Some tweaks to limit the redundancy (you mention frequent
That sounds good, thanks. Updated.
With that blurb about heavy load to tablet servers, I was trying to get an example of conditions where the manifestation of the bug could be seen (election storms). Indeed, I think we can drop that since if removing the mention of tablet servers, the seconds sentence simply re-phrases the same information.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@204
PS1, Line 204: /`NOT NULL` predicates to leverage SIMD instructions
> SGTM
Ack
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@272
PS1, Line 272: vice-versa.
> Ah, you're right. For some reason, one of my commits from a while ago shows
Ack
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@100
PS3, Line 100: m
> nit: the, same below
Done
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@140
PS3, Line 140: initiated
> nit: "initiating", or "that initiated"
Done
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@179
PS3, Line 179: with specified dimension
> nit: "with a dimension"? "with a specific dimension"?
Done
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@212
PS3, Line 212: was contributing to the frequent leader change phenomenon
: (a.k.a. election storms)
> nit: could lead to frequent elections.
As discussed, I removed this sentence.
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@216
PS3, Line 216: case if b
> nit: "in cases where"
Done
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@232
PS3, Line 232: Kudu
> nit: the Kudu
Done
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@240
PS3, Line 240: pending AlterSchema
: transaction.
> nit: plurality is off, "pending `AlterSchema` transactions"? "a pending `Al
Done
http://gerrit.cloudera.org:8080/#/c/14404/3/docs/release_notes.adoc@296
PS3, Line 296: * honeyhexin
> On Slack, she goes by Xin He.
Thanks! Updated.
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
Gerrit-Comment-Date: Thu, 10 Oct 2019 01:47:07 +0000
Gerrit-HasComments: Yes
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello honeyhexin, Attila Bukor, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, Grant Henke, Greg Solovyev, Hao Hao, Volodymyr Verovkin, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14404
to look at the new patch set (#8).
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
WIP [docs] Kudu 1.11 release notes draft
The rendered version is available at:
https://gist.github.com/alexeyserbin/53200325d07f4c922682580489e3d45f
WIP:
* this is published just as working draft to iterate upon
* the branch is wrong (should be in branch-1.11.x): DO NOT COMMIT
into the master branch
Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
---
M docs/release_notes.adoc
1 file changed, 191 insertions(+), 13 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/14404/8
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 8
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14404 )
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
Patch Set 1:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/14404/3//COMMIT_MSG
Commit Message:
http://gerrit.cloudera.org:8080/#/c/14404/3//COMMIT_MSG@7
PS3, Line 7: WIP [docs] Kudu 1.11 release notes draft
> Could you also post to your own github branch and post a link so reviewers
Done
https://gist.github.com/alexeyserbin/53200325d07f4c922682580489e3d45f
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@123
PS1, Line 123:
: * The --master_client_location_assignment_enabled master's flag is now
: available for usage without the necessity to add --unlock_unsafe_flags.
> Actually upon looking at the commits, I thought the default behavior has ch
Ah, maybe getting rid of this item might be the best option :) You are right.
Done.
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
Gerrit-Comment-Date: Thu, 10 Oct 2019 01:52:46 +0000
Gerrit-HasComments: Yes
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello honeyhexin, Attila Bukor, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, Grant Henke, Greg Solovyev, Hao Hao, Volodymyr Verovkin, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14404
to look at the new patch set (#3).
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
WIP [docs] Kudu 1.11 release notes draft
WIP:
* this is published just as working draft to iterate upon
* the branch is wrong: DO NOT COMMIT
Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
---
M docs/release_notes.adoc
1 file changed, 193 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/14404/3
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14404 )
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
Patch Set 6:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/14404/6/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/6/docs/release_notes.adoc@157
PS6, Line 157: * A quick-start example of using Apache Spark to load, query, and modify
> nit: I'd change these two bullet points to active voice
Done
http://gerrit.cloudera.org:8080/#/c/14404/6/docs/release_notes.adoc@188
PS6, Line 188: * Decoding of deltas' timestamps has been optimized
> "Optimized decoding of deltas' timestamps" - I'd use active voice instead o
Done
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 6
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
Gerrit-Comment-Date: Fri, 11 Oct 2019 00:19:54 +0000
Gerrit-HasComments: Yes
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14404 )
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
Patch Set 7:
(22 comments)
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@34
PS7, Line 34: * Flume 1.8+ requires Java 8 at runtime even though the Kudu Flume integration
: is Java 7 compatible. Flume 1.9 is the default dependency version as of
: Kudu 1.9.0.
:
: * Hadoop 3.0+ requires Java 8 at runtime even though the Kudu Hadoop integration
: is Java 7 compatible. Hadoop 3.2 is the default dependency version as of
: Kudu 1.9.0.
Should probably modify these bits too. See KUDU-2099 for more details on the removal of Java 7 support.
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@48
PS7, Line 48: * `TabletLocationsPB.ReplicaPB` has been deprecated since Kudu 1.11.0 in both
: {cpp} and Java Kudu clients. The server side still recognizes the
: `TabletLocationsPB.ReplicaPB` message for backward compatibility with clients
: of prior versions.
While accurate, we should try to describe this in concrete user-facing terms. Is there any? Won't new clients operate correctly against old servers, and vice versa? If so, there's probably nothing worth mentioning here.
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@52
PS7, Line 52:
: * Support for Java 7 has been deprecated since Kudu 1.5.0 and may be removed in
: the next major release.
Java 7 support was removed in 1.10.0. Not sure why this carried over into here.
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@59
PS7, Line 59: While a tablet
: server is in maintenance, its downtime does not automatically trigger
: re-replication of its replicas
Nit: perhaps "While in this mode, the tablet server's replicas will not be rereplicated if it fails. Only upon exiting maintenance will re-replication be triggered for any remaining under-replicated tablets."
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@68
PS7, Line 68: With the built-in NTP
: client enabled
Nit: "When enabled,"
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@69
PS7, Line 69: it's no longer required to keep the system clock of Kudu
: nodes synchronzed with NTP (might be useful in case of containerized
: deployments)
Nit: "system clock synchronization for nodes running Kudu is no longer necessary, though it might still be useful for containerized deployments."
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@71
PS7, Line 71: With the built-in NTP client enabled, the internal wallclock
: is synchronized with NTP servers specified with the `--builtin_ntp_servers`
: flag. By default, the built-in NTP client uses public servers of the
: `pool.ntp.org` project
Nit: "The list of NTP servers to synchronize against is specified with the `--builtin_ntp_servers` flag. By default, Kudu will use public servers hosted by the NTP Pool project."
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@75
PS7, Line 75: `0.pool.ntp.org,1.pool.ntp.org,2.pool.ntp.org,3.pool.ntp.org`
I don't think we need to get this specific; just referencing pool.ntp.org is good enough.
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@75
PS7, Line 75: To switch Kudu
: masters and tablet servers to use the built-in NTP client, add
: `--time_source=builtin` into the set of their flags, customizing the
: `--builtin_ntp_servers` flag if necessary
: (see link:https://issues.apache.org/jira/browse/KUDU-2935[KUDU-2935]).
:
Nit: "To use the built-in NTP client, set `--time_source=builtin` and reconfigure `--builtin_ntp_servers` if necessary (see KUDU-2935)."
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@81
PS7, Line 81: via
: GetTableStatistics master's RPC
Let's refer to specific APIs rather than to RPCs, as users interact with the former but not the latter.
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@87
PS7, Line 87: uuid
Should we surround this with triangular brackets to emphasize that it's not a string literal, but a variable?
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@91
PS7, Line 91: * The `kudu` CLI tool now supports altering and dropping table columns. Using
: the newly introduced `kudu table delete_column` sub-command, it's possible to
: drop a column of the specified table. The newly introduced commands such as
: `kudu table column_set_default`, `kudu table column_remove_default`,
: `kudu table column_set_compression`, `kudu table column_set_encoding`,
: and `kudu table column_set_block_size` allows for altering a column of the
: specified table correspondingly.
Given the asymmetry between these two (unlike the other new CLI tools below), I think this will be clearer if you break it up into one bullet for altering columns (with all of the various subtools) and one for dropping columns.
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@119
PS7, Line 119: tablet server
Probably drop, as it affects masters too.
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@126
PS7, Line 126: Tablet server
Isn't this for the master web ui?
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@129
PS7, Line 129: groups
singular
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@130
PS7, Line 130: that tablet's
replace with "the"
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@133
PS7, Line 133: Kudu
"the Kudu"
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@153
PS7, Line 153: * Tablet servers now consider the available space when selecting data
: directories for tablets and blocks
How about "Tablet servers now consider available disk space when choosing a set of data directories for a tablet's data directory group, and when deciding in which data directory a new block should be written."
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@163
PS7, Line 163: constrains
constraints
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@164
PS7, Line 164: The previous behavior was to reject the whole batch of write
: operations if a violation of the schema constraints is detected even
: for a single row
Do we need to surface this as an upgrade note?
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@169
PS7, Line 169: to
Replace with "with a"
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@170
PS7, Line 170: use
"Use the foo and bar methods of the C++ and Java Kudu clients."
Below too.
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 7
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
Gerrit-Comment-Date: Fri, 11 Oct 2019 04:28:22 +0000
Gerrit-HasComments: Yes
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello honeyhexin, Attila Bukor, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, Grant Henke, Greg Solovyev, Hao Hao, Volodymyr Verovkin, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14404
to look at the new patch set (#6).
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
WIP [docs] Kudu 1.11 release notes draft
The rendered version is available at:
https://gist.github.com/alexeyserbin/53200325d07f4c922682580489e3d45f
WIP:
* this is published just as working draft to iterate upon
* the branch is wrong: DO NOT COMMIT
Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
---
M docs/release_notes.adoc
1 file changed, 187 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/14404/6
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 6
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has abandoned this change. ( http://gerrit.cloudera.org:8080/14404 )
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
Abandoned
Obsoleted by https://gerrit.cloudera.org/#/c/14414/
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: abandon
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 8
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14404 )
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
Patch Set 7:
(22 comments)
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@34
PS7, Line 34: * Flume 1.8+ requires Java 8 at runtime even though the Kudu Flume integration
: is Java 7 compatible. Flume 1.9 is the default dependency version as of
: Kudu 1.9.0.
:
: * Hadoop 3.0+ requires Java 8 at runtime even though the Kudu Hadoop integration
: is Java 7 compatible. Hadoop 3.2 is the default dependency version as of
: Kudu 1.9.0.
> Should probably modify these bits too. See KUDU-2099 for more details on th
Ah, great. I removed this sice Java8 is already a requirement in Kudu 1.10.0
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@48
PS7, Line 48: * `TabletLocationsPB.ReplicaPB` has been deprecated since Kudu 1.11.0 in both
: {cpp} and Java Kudu clients. The server side still recognizes the
: `TabletLocationsPB.ReplicaPB` message for backward compatibility with clients
: of prior versions.
> While accurate, we should try to describe this in concrete user-facing term
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@52
PS7, Line 52:
: * Support for Java 7 has been deprecated since Kudu 1.5.0 and may be removed in
: the next major release.
> Java 7 support was removed in 1.10.0. Not sure why this carried over into h
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@59
PS7, Line 59: While a tablet
: server is in maintenance, its downtime does not automatically trigger
: re-replication of its replicas
> Nit: perhaps "While in this mode, the tablet server's replicas will not be
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@68
PS7, Line 68: With the built-in NTP
: client enabled
> Nit: "When enabled,"
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@69
PS7, Line 69: it's no longer required to keep the system clock of Kudu
: nodes synchronzed with NTP (might be useful in case of containerized
: deployments)
> Nit: "system clock synchronization for nodes running Kudu is no longer nece
The idea was to mention that the built-in NTP client might be useful for containerized deployments.
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@71
PS7, Line 71: With the built-in NTP client enabled, the internal wallclock
: is synchronized with NTP servers specified with the `--builtin_ntp_servers`
: flag. By default, the built-in NTP client uses public servers of the
: `pool.ntp.org` project
> Nit: "The list of NTP servers to synchronize against is specified with the
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@75
PS7, Line 75: `0.pool.ntp.org,1.pool.ntp.org,2.pool.ntp.org,3.pool.ntp.org`
> I don't think we need to get this specific; just referencing pool.ntp.org i
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@75
PS7, Line 75: To switch Kudu
: masters and tablet servers to use the built-in NTP client, add
: `--time_source=builtin` into the set of their flags, customizing the
: `--builtin_ntp_servers` flag if necessary
: (see link:https://issues.apache.org/jira/browse/KUDU-2935[KUDU-2935]).
:
> Nit: "To use the built-in NTP client, set `--time_source=builtin` and recon
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@81
PS7, Line 81: via
: GetTableStatistics master's RPC
> Let's refer to specific APIs rather than to RPCs, as users interact with th
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@87
PS7, Line 87: uuid
> Should we surround this with triangular brackets to emphasize that it's not
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@91
PS7, Line 91: * The `kudu` CLI tool now supports altering and dropping table columns. Using
: the newly introduced `kudu table delete_column` sub-command, it's possible to
: drop a column of the specified table. The newly introduced commands such as
: `kudu table column_set_default`, `kudu table column_remove_default`,
: `kudu table column_set_compression`, `kudu table column_set_encoding`,
: and `kudu table column_set_block_size` allows for altering a column of the
: specified table correspondingly.
> Given the asymmetry between these two (unlike the other new CLI tools below
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@119
PS7, Line 119: tablet server
> Probably drop, as it affects masters too.
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@126
PS7, Line 126: Tablet server
> Isn't this for the master web ui?
Whoops, indeed. Good catch!
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@129
PS7, Line 129: groups
> singular
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@130
PS7, Line 130: that tablet's
> replace with "the"
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@133
PS7, Line 133: Kudu
> "the Kudu"
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@153
PS7, Line 153: * Tablet servers now consider the available space when selecting data
: directories for tablets and blocks
> How about "Tablet servers now consider available disk space when choosing a
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@163
PS7, Line 163: constrains
> constraints
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@164
PS7, Line 164: The previous behavior was to reject the whole batch of write
: operations if a violation of the schema constraints is detected even
: for a single row
> Do we need to surface this as an upgrade note?
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@169
PS7, Line 169: to
> Replace with "with a"
Done
http://gerrit.cloudera.org:8080/#/c/14404/7/docs/release_notes.adoc@170
PS7, Line 170: use
> "Use the foo and bar methods of the C++ and Java Kudu clients."
Done
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 7
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
Gerrit-Comment-Date: Fri, 11 Oct 2019 08:10:24 +0000
Gerrit-HasComments: Yes
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14404 )
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
Patch Set 2:
(28 comments)
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc
File docs/release_notes.adoc:
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@51
PS1, Line 51: 11
I think it was deprecated in 1.10
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@51
PS1, Line 51: TabletLocationsPB.ReplicaPB
nit: wrap with ``s so it shows up as monospaced code font.
Same with gflag and other object names below.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@63
PS1, Line 63: * TODO Kudu now supports maintenance mode
: (see link:https://issues.apache.org/jira/browse/KUDU-2069[KUDU-2069]).
Kudu now supports putting tablet servers into maintenance. While a tablet server is in maintenance, its downtime will not automatically trigger re-replication of its replicas. The `kudu tserver state enter_maintenance` and `kudu tserver state exit_maintenance` tools are added to orchestrate tablet server maintenance, and the `kudu tserver list` tool is amended with a "state" column option to display current state of each tablet server.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@66
PS1, Line 66: b
nit: a built-in
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@66
PS1, Line 66: *
Might be worth trying to describe what the user should expect if switching over to the built-in client. Faster NTP convergence? No system privileges required? Anything else?
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@73
PS1, Line 73: f their run-time flags
This might be confusing since it might make it seem like the flag is tagged "runtime".
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@74
PS1, Line 74: The initial implementation
: of the built-in NTP client is not RFC-compliant at this point, but it is good
: enough to work with well-behaved NTP servers. For details on further
: improvements, see the sub-tasks of the
nit: This seems like a lot of detail for a release note. Maybe we should add docs for this and point the release notes at the docs? Perhaps in the Configuring Kudu page?
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@82
PS1, Line 82: can
nit: "will" so it's obvious that this isn't optional. Or "can now optionally" if it is
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@83
PS1, Line 83: ar
are
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@97
PS1, Line 97:
nit: `kudu table` tool? Or say that new tools `kudu table get_extra_configs` and `kudu table set_extra_config` are introduced. That way the tools are more easily discovered.
Same with `drop_range_partition`
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@111
PS1, Line 111:
nit: CFileReaders (plural), and maybe we wrap these in quotes. I.e. as "CFileReaders" and "BloomFileReaders"
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@113
PS1, Line 113: * The memory consumed by CFileReaders and BloomFileReaders is factored out and
: accounted separately by the tablet server memory tracking. The stats are
: available via tablet server Web UI as CFileReader and BloomFileReaders
: entries.
Not sure this deserves a release note unless there are user-facing ways to leverage this.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@123
PS1, Line 123: (see link:https://issues.apache.org/jira/browse/KUDU-1561[KUDU-1561]).
:
: * Tablet server Web UI now supports sorting the list of tables by the col
We should note that this has been enabled by default and maybe drop the mention of `--unlock_unsafe_flags`?
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@127
PS1, Line 127:
nit: "will now" so it's obvious this is new non-optional behavior
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@133
PS1, Line 133:
nit: "in case no replica is local"
Also maybe note the previous behavior this replaces? Or why this is beneficial? I believe it's to prevent hotspotting?
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@135
PS1, Line 135:
: * For scan operations run with CLOSEST_REPLICA selection mode, Kudu Java client
: now picks a random available replica in case if no replica is local
Maybe, "The serialization of in-memory rows to Kudu's wire format has been optimized to be more CPU efficient."?
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@144
PS1, Line 144: now possib
nit: capitalize these
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@159
PS1, Line 159: ed.
nit: previous? Unless this behavior can still be achieved somehow
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@163
PS1, Line 163: client. The legacy behavior was to reject the whole batch of write operation
Does this happen by default? If so, use "will now" instead of "can", and if not, maybe "can now optionally be placed..." and mentioned how?
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@164
PS1, Line 164: tion of the schema constraints is detected even
I don't think this is the right link
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@181
PS1, Line 181: * Optimized the initialization of DeltaMemStore for the case when no matching
: deltas are present
: (see link:https://issues.apache.org/jira/browse/KUDU-2381[KUDU-2381]).
Should this be classified as a bug fix?
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@185
PS1, Line 185: ory reserved by tcmall
nit: drop
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@188
PS1, Line 188:
nit: drop
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@189
PS1, Line 189: Scan tokens are now rehydrated in a more robus
nit: "and NULL/NOT NULL predicates to leverage SIMD operations"
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@196
PS1, Line 196:
: [[rn_1.11.0_fixed_issues]]
: == Fixed Issues
:
: * Fixed an issue when a follower replica was granting `yes` vote right after
: processing recent Raft transaction f
nit: maybe cut down some of the specifics; also not specific to tablet servers, right?:
"Fixed an issue with Raft voting on nodes with slow WAL disks that could lead to frequent leader changes when those nodes were under heavy load."
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@204
PS1, Line 204: frequent leader change phenomenon (a.k.a. election storms)
nit: maybe just:
"Fixed a tablet server crash that may have surfaced after a disk failure."?
I don't imagine "orphaned blocks" being particularly meaningful to users.
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@207
PS1, Line 207:
nit: Fixed a crash in
http://gerrit.cloudera.org:8080/#/c/14404/1/docs/release_notes.adoc@272
PS1, Line 272: [[rn_1.11.0_known_issues]]
Should note the first-time contributors. I think that's:
- Forec (Yao Wang)
- hannahvnguyen (Hannah Nguyen)
- iRitwik (Ritwik Yadav)
- lingbin
- Volodymyr Verovkin
- Xiaokai Wang
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>
Gerrit-Comment-Date: Wed, 09 Oct 2019 21:09:05 +0000
Gerrit-HasComments: Yes
[kudu-CR] WIP [docs] Kudu 1.11 release notes draft
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello honeyhexin, Attila Bukor, Kudu Jenkins, Andrew Wong, Yao Xu, Adar Dembo, Grant Henke, Greg Solovyev, Hao Hao, Volodymyr Verovkin, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14404
to look at the new patch set (#2).
Change subject: WIP [docs] Kudu 1.11 release notes draft
......................................................................
WIP [docs] Kudu 1.11 release notes draft
WIP:
* this is published just as working draft to iterate upon
* the branch is wrong: DO NOT COMMIT
Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
---
M docs/release_notes.adoc
1 file changed, 170 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/14404/2
--
To view, visit http://gerrit.cloudera.org:8080/14404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I201f4e07504d59e668b99d6154c9412ed79c9d35
Gerrit-Change-Number: 14404
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gs...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Volodymyr Verovkin <ve...@cloudera.com>
Gerrit-Reviewer: Yao Xu <oc...@gmail.com>
Gerrit-Reviewer: honeyhexin <ho...@sohu.com>