Yingchun Lai has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19512 )

Change subject: [docs] Add release notes for 1.17.0
......................................................................


Patch Set 10:

(29 comments)

http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc
File docs/release_notes.adoc:

http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@195
PS10, Line 195: tool
> nit: drop 'tool' for brevity
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@239
PS10, Line 239: before Kudu 1.12
> replace with:
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@241
PS10, Line 241: flag
> flagging
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@248
PS10, Line 248: by `--show_slow_scans`
> by tweaking the `--show_slow_scans` flag for tablet servers
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@256
PS10, Line 256: It's no longer necessary
> It's not necessary
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@260
PS10, Line 260: to build and run
> building and running
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@265
PS10, Line 265: Fixed an issue where historical MVCC data older than the 
ancient history mark (configured by
              : setting `--tablet_history_max_age_sec`) which has only delete 
operations without any update
              : operations hasn't been compacted correctly, then the ancient 
history data can not be GCed if the
              : tablet is created before Kudu 1.10 at which version doesn’t 
support live row counting
> Fixed an issue where historical MVCC data older than the ancient history ma
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@271
PS10, Line 271: crashes
> could potentially crash
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@273
PS10, Line 273: Fixed a hang bug in the Kudu tablet server that is started by a 
user who has no permission of the
              : tablet metadata file
> Fixed a bug when a Kudu tablet server started under an OS account that had
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@276
PS10, Line 276: doesn’t
> would not
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@278
PS10, Line 278: doesn’t
> would not
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@280
PS10, Line 280: Fixed a bug in the Java client where reading repetitive rows 
even if in fault tolerant scan mode
> Fixed a bug in the Java client where under certain conditions same rows wou
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@282
PS10, Line 282: don't
              : update
> would not be updated
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@285
PS10, Line 285: can't
> would not
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@287
PS10, Line 287: where Hive failed to query on Kudu tables
> that was causing failures when scanning Kudu tables from Hive
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@290
PS10, Line 290: CLI tool `kudu table copy` failed to copy
> ... the `kudu table copy` CLI tool would fail copying ...
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@292
PS10, Line 292: CLI tool `kudu master unsafe_rebuild` failed to rebuild the 
correct latest
              : schema when the table to be rebuilt is unhealthy
> ... the `kudu master unsafe_rebuild` CLI tool would rebuild the system cata
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@295
PS10, Line 295: varchar
> nit: VARCHAR
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@295
PS10, Line 295: string
> nit: STRING
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@295
PS10, Line 295: binary
> nit: BINARY
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@295
PS10, Line 295: which have
> that had
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@299
PS10, Line 299: that `kudu table copy` crashes
> of the `kudu table copy` CLI tool crashing
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@299
PS10, Line 299: when encountered an error when copy rows to the
              : destination table,
> if encountering an error while copying rows to the destination table.
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@300
PS10, Line 300: now it will exit gracefully and leave some helpful logs.
> Whoops, a typo.  It should have been:
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@302
PS10, Line 302: crashes if the `--list_detail` flag is
              : enabled
> would crash if the `--list_detail` flag was enabled
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@305
PS10, Line 305: of the Ranger client subprocess on Kudu master when receiving a 
response exceeds the
              : maximum message size, now it's able to clear the communication 
channel to receive next messages from
              : the subprocess
> when a sub-process running Ranger client would crash when receiving a overs
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@309
PS10, Line 309: of a crash with SIGILL signal when attempt to run an 
application linked with
              : kudu_client library at a machine lacking SSE4.2 support
> when a Kudu application linked with kudu_client library would crash with SI
Done


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@343
PS10, Line 343:  If not, use new APIs introduced by Kudu 1.17.
> I'm not sure why to add this -- this is quite strange.  If we are declaring
Wenzhe Zhou gave me some suggestions in PS1, "Applications which call the new 
Java APIs added in Kudu 1.17 cannot be compiled against Kudu 1.16 client 
library.", the "vice-versa" is not accurate.


http://gerrit.cloudera.org:8080/#/c/19512/10/docs/release_notes.adoc@349
PS10, Line 349:  If not, use new APIs introduced by Kudu 1.17.
> Ditto: why to add this if the prior sentence is true?
Same.



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.17.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I4381e7824dcf7f500165aa3af9dca09a86ac1dd9
Gerrit-Change-Number: 19512
Gerrit-PatchSet: 10
Gerrit-Owner: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Andrew Wong <andrew.w...@cloudera.com>
Gerrit-Reviewer: Ashwani Raina <ara...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Hongjiang Zhang <hongjizh...@ebay.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: KeDeng <kdeng...@gmail.com>
Gerrit-Reviewer: Khazar Mammadli <mammadli.kha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Kurt Deschler <kdesc...@cloudera.com>
Gerrit-Reviewer: Mahesh Reddy <mre...@cloudera.com>
Gerrit-Reviewer: Marton Greber <greber...@gmail.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>
Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com>
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Reviewer: Yuqi Du <shenxingwuy...@gmail.com>
Gerrit-Reviewer: Zoltan Chovan <zcho...@cloudera.com>
Gerrit-Reviewer: bsglz <18031...@qq.com>
Gerrit-Reviewer: harinreddy <hnre...@us.ibm.com>
Gerrit-Reviewer: yejiabao <yejia...@huawei.com>
Gerrit-Reviewer: zhanghongbin <1079039...@qq.com>
Gerrit-Reviewer: Ádám Bakai <aba...@cloudera.com>
Gerrit-Comment-Date: Tue, 20 Jun 2023 04:23:38 +0000
Gerrit-HasComments: Yes

Reply via email to