Todd Lipcon has posted comments on this change. Change subject: Release notes for 0.7.1 ......................................................................
Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/2446/1/docs/release_notes.adoc File docs/release_notes.adoc: Line 66: a tablet was set to be deleted while participating in remote boostrap at the same time. I think we should use more user-facing terminology here that explains in what cases this happens. For example in this case: KUDU-1325 fixes a tablet server crash that could occur during table deletion. In some cases, while a table was being deleted, other replicas would attempt to re-replicate tablets to servers that had already processed the deletion. This could trigger a race condition that caused a crash. Line 69: a row is deleted, re-inserted, and updated. I think we should make this a little more specific: fixes a potential data corruption and crash that could shortly after tablet server restarts in workloads that repeatedly delete and re-insert rows with the same primary key. In most cases, this corruption affected only a single replica and could be repaired by re-replicating from another. Line 73: another. again can we make this more user-focused? in particular this would affect any scans using the java client that read large numbers of rows from multi-tablet tables. (batching is kind of an implementation detail) Line 76: hybrid clock could start running backwards, resulting in a crash followed by an inability to I think we should say "jump backwards" instead of "start running" Line 80: where null values weren't handled properly. which prevented reading rows with NULL values -- To view, visit http://gerrit.cloudera.org:8080/2446 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75e32208754ce6e0d026d097871ccc05d2ca0cf7 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Misty Stanley-Jones <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
