Alexey Serbin 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:

(26 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@46
PS6, Line 46: more scan resistant
more resistant


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


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

  master's and tablet server's

or

  masters' and tablet servers'


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

NOTE: it's better to specify more digits/symbols when referring git UUIDs to 
avoid conflicts; the widespread best practice is to use 9 symbols


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


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 
value greater than ...


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


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 
tablet server bootstrap after switching --rowset_metadata_store_keys to 'true'.


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


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


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


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


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


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


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


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 
servers


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


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


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


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


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


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 --> 
Fixed


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


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


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 
natively (see 1351c19a9 for details).

Instead, please add corresponding note for the changelist mentioned above.  It 
might be something like below:

  * SSE2 and AVX code now uses native NEON instructions on ARM64/AArch64


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 between 
consecutive write operations within the same KuduSession.



--
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: Fri, 15 Nov 2024 20:11:58 +0000
Gerrit-HasComments: Yes

Reply via email to