Adar Dembo has posted comments on this change. Change subject: KUDU-1296. Release notes for 0.7.0 ......................................................................
Patch Set 1: (10 comments) http://gerrit.cloudera.org:8080/#/c/2062/1/docs/release_notes.adoc File docs/release_notes.adoc: Line 62: .. The C++ client has a new scanner API. The old way of using `vector<KuduRowResult>` is Nit: "old API of vector..." is more precise than "old way of using vector...". Line 66: affected. Code that does note change this parameter when creating new tables and that assumes not, not note. But more precisely: Applications that do not change this parameter during table creation and assume that only replica is created may not work properly if only 1-2 tablet servers are available. Line 70: and Java clients. It's is now recommended for use. Hmm, so what was the state of affairs earlier? Maybe say something like "it is no longer considered experimental" or "it is now ready for production", or something like that? Line 72: .. The foundation for a Spark integration was added. It contains a simple `kuduRDD` that wraps Nit: "a Spark integration" sounds a little weird. Is 'a' really necessary? Line 80: .. An important file descriptor leak was fixed, it was triggered when tablets were deleted and Nit: An important file descriptor leak triggered when tablets were deleted was fixed. I don't think the restarted part is necessary; it's obvious that if you restart a process it lets go of its old fds. Line 82: .. Inflight batches in the Java client could get stuck if the previous one encountered an error. Nit (singular/plural confusion): An inflight batch in the Java client could get stuck if the previous batch encountered an error. Line 84: .. Performance for the file block manager was improved but it's still not recommended to use it. Nit: still not recommended for production use. Line 85: .. The Master's catalog manager now tries to spread the tablets more evenly. This doesn't affect Nit: catalog manager is an implementation detail. I'd say: "The master now tries to spread the tablets more evenly across the cluster during table creation. This doesn't affect existing tables." Line 87: .. The licensing documents were changed in order to follow ASF guidelines. Nit: were changed to adhere to ASF guidelines. Line 88: .. Building Kudu can now be done out-of-tree but by default it still expects the "blessed There's not enough context for this to be useful information, and the release notes would be quite long with sufficient context. I'd say "Kudu now requires an out-of-tree build directory. Review the build instructions for additional recommendations". -- 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: 1 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
