From 46a477d99e357850fa4533d5f935d3b812e2f90e Mon Sep 17 00:00:00 2001 From: Amitanand Aiyer Date: Mon, 13 Apr 2020 13:10:03 -0700 Subject: [PATCH] [#4211] Retain delete markers on index tables only during backfill Summary: Fix negation during ShouldRetainDeleteMarkers Retain delete markers on index tables only during backfill Issue introduced by 7c46f82e616a57ed2aa3db166422fb77be2cec69 Test Plan: TBD. add unit test Reviewers: bogdan, mikhail Reviewed By: mikhail Subscribers: kannan, ybase Differential Revision: https://phabricator.dev.yugabyte.com/D8282 --- src/yb/tablet/tablet_retention_policy.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/yb/tablet/tablet_retention_policy.cc b/src/yb/tablet/tablet_retention_policy.cc index 21b7bb6d369c..01337711d7fd 100644 --- a/src/yb/tablet/tablet_retention_policy.cc +++ b/src/yb/tablet/tablet_retention_policy.cc @@ -108,7 +108,7 @@ void TabletRetentionPolicy::UnregisterReaderTimestamp(HybridTime timestamp) { bool TabletRetentionPolicy::ShouldRetainDeleteMarkersInMajorCompaction() const { // If the index table is in the process of being backfilled, then we // want to retain delete markers until the backfill process is complete. - return !metadata_.schema().table_properties().IsBackfilling(); + return metadata_.schema().table_properties().IsBackfilling(); } HybridTime TabletRetentionPolicy::HistoryCutoffToPropagate(HybridTime last_write_ht) {