Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9069 )
Change subject: KUDU-2126: Add conditional check to prevent unnecessary fsyncs ...................................................................... KUDU-2126: Add conditional check to prevent unnecessary fsyncs This patch includes a unit test to detect fsyncs that happened when a DeleteTablet RPC tries to tombstone an already tombstoned tablet. In order to fix the issue, this patch adds additional checking before executing another DeleteTabletData that leads to additional fsyncs. Change-Id: Idde8b413374f43ca2ef09339f0f412208f03685e Reviewed-on: http://gerrit.cloudera.org:8080/9069 Reviewed-by: Mike Percy <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M src/kudu/integration-tests/delete_tablet-itest.cc M src/kudu/tablet/tablet_metadata.cc M src/kudu/tablet/tablet_metadata.h M src/kudu/tserver/ts_tablet_manager.cc 4 files changed, 69 insertions(+), 0 deletions(-) Approvals: Mike Percy: Looks good to me, approved Kudu Jenkins: Verified Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9069 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Idde8b413374f43ca2ef09339f0f412208f03685e Gerrit-Change-Number: 9069 Gerrit-PatchSet: 4 Gerrit-Owner: Jeffrey F. Lukman <[email protected]> Gerrit-Reviewer: Jeffrey F. Lukman <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
