Adar Dembo has posted comments on this change. Change subject: KUDU-2055: Coalesce hole punching when deleting groups of blocks ......................................................................
Patch Set 3: I'd like to avoid the mega-review that we had for the other patch by carving this one up. Here are some ideas for subpatches: 1. Patch to add BlockDeletionTransaction and plumb it through the LBM. At this stage it's just a new abstraction and doesn't yet change semantics or improve performance. 2. Patch to introduce (and test) coalescing of intervals, in fairly generic fashion. 3. Patch to add coalescing support to BlockDeletionTransaction. If this is tiny, maybe combine it with patch #2. 4. Patch to start using BlockDeletionTransaction outside of the 'fs' module. -- To view, visit http://gerrit.cloudera.org:8080/7656 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iecb252b3a5665d5471bb82301d0c8012a68de959 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-HasComments: No
