kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ale...@apache.org
Subject kudu git commit: TSTabletManager::DeleteTabletData() should log delete_type on success
Date Fri, 09 Feb 2018 01:10:21 GMT
Repository: kudu
Updated Branches:
  refs/heads/master 9b1281c95 -> f5d7292fa


TSTabletManager::DeleteTabletData() should log delete_type on success

Change-Id: I5520334b86a540e6df64fa7036a7a83a7db473ac
Reviewed-on: http://gerrit.cloudera.org:8080/9259
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <aserbin@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/f5d7292f
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/f5d7292f
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/f5d7292f

Branch: refs/heads/master
Commit: f5d7292fa70a0a6a5461d419c187a5e65354d404
Parents: 9b1281c
Author: Mike Percy <mpercy@apache.org>
Authored: Thu Feb 8 14:34:52 2018 -0800
Committer: Alexey Serbin <aserbin@cloudera.com>
Committed: Fri Feb 9 01:04:50 2018 +0000

----------------------------------------------------------------------
 src/kudu/tserver/ts_tablet_manager.cc | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/f5d7292f/src/kudu/tserver/ts_tablet_manager.cc
----------------------------------------------------------------------
diff --git a/src/kudu/tserver/ts_tablet_manager.cc b/src/kudu/tserver/ts_tablet_manager.cc
index 1d3abfc..e96c7cf 100644
--- a/src/kudu/tserver/ts_tablet_manager.cc
+++ b/src/kudu/tserver/ts_tablet_manager.cc
@@ -1280,8 +1280,10 @@ Status TSTabletManager::DeleteTabletData(
   RETURN_NOT_OK(meta->DeleteTabletData(delete_type, last_logged_opid));
   last_logged_opid = meta->tombstone_last_logged_opid();
   LOG(INFO) << LogPrefix(tablet_id, meta->fs_manager())
-            << "tablet deleted: last-logged OpId: "
-            << (last_logged_opid ? OpIdToString(*last_logged_opid) : "(unknown)");
+            << "tablet deleted with delete type "
+            << TabletDataState_Name(delete_type) << ": "
+            << "last-logged OpId "
+            << (last_logged_opid ? OpIdToString(*last_logged_opid) : "unknown");
   MAYBE_FAULT(FLAGS_fault_crash_after_blocks_deleted);
 
   CHECK_OK(Log::DeleteOnDiskData(meta->fs_manager(), tablet_id));


Mime
View raw message