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:

(36 comments)

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

http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@40
PS6, Line 40: 3.5
> 3.5 inclusive
Yes, updated.


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@46
PS6, Line 46: more scan resistant
> more resistant
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@47
PS6, Line 47: in
> from ?
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@55
PS6, Line 55: master’s and tablet servers’
> nit: use plural or singular for the both
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@75
PS6, Line 75: timeouts: this
> nit: Replace with "timeouts. This"
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@77
PS6, Line 77: timing: see
> nit: Replace with "timing. See"
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@79
PS6, Line 79: servers: this
> nit: Replace with "servers. This"
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@92
PS6, Line 92: commit
> changelist 3bdaf50b5
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@100
PS6, Line 100: set
             :   tserver_send_tombstoned_tablets_report_inteval_secs to a value 
greater than
> ... tserver_send_tombstoned_tablets_report_inteval_secs should be set to a
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@104
PS6, Line 104: flush DRS or MRS
> DRS or MRS flush
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@108
PS6, Line 108: Support for updating min/max encoded keys at bootstrapping to 
accelerate scanning speed has been
             :   added
> Previously absent min/max keys in rowset metadata are now populated during
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@110
PS6, Line 110: (
> nit: add a space before the parenthesis
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@111
PS6, Line 111: is disabled
> is now disabled
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@115
PS6, Line 115: runs
> now runs
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@124
PS6, Line 124: Soft delete support
> Support for table soft-deletion
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@128
PS6, Line 128: gradle
> Gradle
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@130
PS6, Line 130: spnego
> SPNEGO
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@132
PS6, Line 132: tablets
> tables
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@132
PS6, Line 132: (number of requested replicas of the tablet
             :   is greater than the number of tablet servers)
> when the number of requested replicas is greater than the number of tablet
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@135
PS6, Line 135: copy
> copying
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@137
PS6, Line 137: rpc_max_message_size
> maximum size of RPC messages
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@137
PS6, Line 137: in C++ client
> via the Kudu C++ client API
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@150
PS6, Line 150: errors writing
> errors while writing
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@152
PS6, Line 152: authz
> authorization
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@181
PS6, Line 181: default: this
> default. This
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@184
PS6, Line 184: invalid
> a no-longer-existing
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@186
PS6, Line 186: Kudu CLI’s
> The Kudu CLI tools
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@186
PS6, Line 186: can
> can now
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@188
PS6, Line 188: could have caused
> could cause
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@189
PS6, Line 189: not getting kicked in
> to not kick in
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@194
PS6, Line 194: Fix
> change here and below for consistency with the rest of the contents: Fix --
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@196
PS6, Line 196: scanner expired
> `Scanner not found`
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@207
PS6, Line 207: kerberos
> Kerberos
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@210
PS6, Line 210: S
> add a space before 'endpoints'
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@213
PS6, Line 213: * Upgraded sse2neon.h to fix the build failure on Ubuntu 20 ARM
             :   (see 
https://issues.apache.org/jira/browse/KUDU-3475[KUDU-3475]).
> This is irrelevant since SSE and AVX arm64 instructions are now implemented
Done


http://gerrit.cloudera.org:8080/#/c/22013/6/docs/release_notes.adoc@217
PS6, Line 217: Kudu handles table schema change between write operations in 
Java client
> Kudu Java client now properly handles concurrent table schema updates betwe
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: Thu, 17 Apr 2025 19:05:36 +0000
Gerrit-HasComments: Yes

Reply via email to