Todd Lipcon has posted comments on this change.

Change subject: KUDU-1296. Release notes for 0.7.0
......................................................................


Patch Set 2:

(8 comments)

http://gerrit.cloudera.org:8080/#/c/2062/2/docs/release_notes.adoc
File docs/release_notes.adoc:

Line 59: The 0.7.0 release is the first one done following the ASF guidelines. 
Here are the main changes:
- "following the ASF guidelines" -> "as part of the Apache Incubator"? makes us 
sound sort of like previous releases were super sketchy or something :)

- maybe also say "and includes a number of fixes and improvements"


Line 63: deprecated, developers should now use `KuduScanBatch`. This change 
improves cases where a
should be a ';' after 'deprecated'. or a '--'.


Line 67: only 1 replica is created may not work properly if only 1-2 tablet 
servers are available.
say that the master may be configured with --default_num_replicas=1 to restore 
the previous default


Line 76: `AUTO_FLUSH_BACKGROUND`, use `countPendingErrors()` and 
`getPendingErrors()` on `KuduSession`
start a new sentence with 'use' (or change to a ':')


Line 80: .. An important file descriptor leak was fixed, it was triggered when 
tablets were deleted and
same deal. period, not comma


Line 82: .. An inflight batch in the Java client could get stuck if the 
previous batch encountered an error.
should say "Fixed a case where"


Line 84: .. Performance for the file block manager was improved but it's still 
not recommended for
comma before 'but'


Line 87: This doesn't affect existing tables.
add 'This will improve the speed at which under-replicated tablets are 
re-replicated after a failure.'


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia3fa0aeffed578c64869facd6fc7aeee216bca3e
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to