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

Change subject: [docs] Add release notes for Kudu 1.17.1
......................................................................


Patch Set 2:

(23 comments)

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

PS1:
> Does it make sense to mention the following changes as well?
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@38
PS1, Line 38: 1.17.1
> 1.17.1 ?
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@40
PS1, Line 40: an unboun
> an unbounded
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@41
PS1, Line 41: schema
> nit: schema
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@46
PS1, Line 46: n
> nit: Not sure what this is, an extra space maybe? I see it after a few of t
Yes all the trailing spaces have been addressed.


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@46
PS1, Line 46: index out of index boun
> nit: index out of bounds
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@49
PS1, Line 49: when system time
> ... when system time is being synchronized ...
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@49
PS1, Line 49:
> nit for here and below: remove the trailing spaces
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@52
PS1, Line 52: encryption-at-rest
> encryption-at-rest
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@55
PS1, Line 55: tablet re
> tablet replica placement
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@55
PS1, Line 55: range-aware
> nit: range-aware
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@58
PS1, Line 58: Fixed NPE thrown by Kudu Java client when establishin
> Fixed NPE thrown by the Java Kudu client when establishing a connection to
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@60
PS1, Line 60:
> nit: drop 'dynamically'?
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@63
PS1, Line 63:
> gauges
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@63
PS1, Line 63:
> Don not expose non-arithmetic gauges via Prometheus metrics
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@66
PS1, Line 66:
> usage
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@69
PS1, Line 69:
> IN-list
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@70
PS1, Line 70: e
> nit: remove the extra space
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@72
PS1, Line 72:
> unexpectedly setting a newly added column cell to its default value when a
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@75
PS1, Line 75: .org/jira/
> nit: an unexpected
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@78
PS1, Line 78: jira/brows
> nit: well-known
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@78
PS1, Line 78: ee https://issues.apache
> Upgraded Netty to version 4.1.110.Final, addressing ...
Done


http://gerrit.cloudera.org:8080/#/c/21828/1/docs/release_notes.adoc@80
PS1, Line 80: * Upgraded Netty
> This should have become
Done



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.17.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I0e825c3ea3f63a5b8a11820a14b33f9b0bc950d1
Gerrit-Change-Number: 21828
Gerrit-PatchSet: 2
Gerrit-Owner: Abhishek Chennaka <achenn...@cloudera.com>
Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ara...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: KeDeng <kdeng...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <mre...@cloudera.com>
Gerrit-Reviewer: Marton Greber <greber...@gmail.com>
Gerrit-Reviewer: Song Jiacheng <songjiach...@thinkingdata.cn>
Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com>
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Reviewer: Zoltan Chovan <zcho...@cloudera.com>
Gerrit-Reviewer: Zoltan Martonka <zmarto...@cloudera.com>
Gerrit-Reviewer: Ádám Bakai <aba...@cloudera.com>
Gerrit-Comment-Date: Mon, 23 Sep 2024 20:42:11 +0000
Gerrit-HasComments: Yes

Reply via email to