Abhishek Chennaka has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22013 )

Change subject: [docs] Update Kudu 1.18.0 release notes
......................................................................


Patch Set 6:

(9 comments)

http://gerrit.cloudera.org:8080/#/c/22013/1/docs/release_notes.adoc
File docs/release_notes.adoc:

http://gerrit.cloudera.org:8080/#/c/22013/1/docs/release_notes.adoc@47
PS1, Line 47:   would otherwise evict frequently accessed data in the cache
> +1
Great, thanks!


http://gerrit.cloudera.org:8080/#/c/22013/5/docs/release_notes.adoc
File docs/release_notes.adoc:

http://gerrit.cloudera.org:8080/#/c/22013/5/docs/release_notes.adoc@56
PS5, Line 56: `Auto-Incrementin
> nit: auto-incrementing
Added the needed formatting from 1.17.0 release notes


http://gerrit.cloudera.org:8080/#/c/22013/5/docs/release_notes.adoc@58
PS5, Line 58: `Auto-Incrementin
> nit: auto-incrementing
Same as above


http://gerrit.cloudera.org:8080/#/c/22013/5/docs/release_notes.adoc@75
PS5, Line 75: identif
> nit: identifying
Done


http://gerrit.cloudera.org:8080/#/c/22013/5/docs/release_notes.adoc@110
PS5, Line 110: KuduSession
> nit: KuduSession
Done


http://gerrit.cloudera.org:8080/#/c/22013/5/docs/release_notes.adoc@113
PS5, Line 113: alter table
> nit: alter table
Done


http://gerrit.cloudera.org:8080/#/c/22013/5/docs/release_notes.adoc@158
PS5, Line 158: 3515
> wrong jira, should be https://issues.apache.org/jira/browse/KUDU-3515
Done


http://gerrit.cloudera.org:8080/#/c/22013/5/docs/release_notes.adoc@178
PS5, Line 178: https://issues.apache.org/jira/browse/KUDU-3570
> wrong jira, should be https://issues.apache.org/jira/browse/KUDU-3570
Good catch. Done


http://gerrit.cloudera.org:8080/#/c/22013/5/docs/release_notes.adoc@190
PS5, Line 190: keytab
> nit: keytab
Done



--
To view, visit http://gerrit.cloudera.org:8080/22013
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.18.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I57642af58d86de0d8f8bc26d06c4607fac6ea321
Gerrit-Change-Number: 22013
Gerrit-PatchSet: 6
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: KeDeng <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Song Jiacheng <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Yuqi Du <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
Gerrit-Reviewer: Ádám Bakai <[email protected]>
Gerrit-Comment-Date: Tue, 12 Nov 2024 22:28:25 +0000
Gerrit-HasComments: Yes

Reply via email to