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 <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Mon, 17 Jun 2019 18:31:35 +0000
Gerrit-HasComments: Yes

Reply via email to