From: Radosław Zarzyński <rzarzyns@redhat.com>
Date: Tue, 13 Jun 2023 10:54:26 +0000 (+0200)
Subject: osd: drop the unused ECTransaction::requires_overwrite()
X-Git-Tag: v19.0.0~841^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F52025%2Fhead;p=ceph.git

osd: drop the unused ECTransaction::requires_overwrite()

Signed-off-by: Radosław Zarzyński <rzarzyns@redhat.com>
---

diff --git a/src/osd/ECTransaction.cc b/src/osd/ECTransaction.cc
index 44876b3a89fa..f74e6c74ade3 100644
--- a/src/osd/ECTransaction.cc
+++ b/src/osd/ECTransaction.cc
@@ -93,19 +93,6 @@ void encode_and_write(
   }
 }
 
-bool ECTransaction::requires_overwrite(
-  uint64_t prev_size,
-  const PGTransaction::ObjectOperation &op) {
-  // special handling for truncates to 0
-  if (op.truncate && op.truncate->first == 0)
-    return false;
-  return op.is_none() &&
-    ((!op.buffer_updates.empty() &&
-      (op.buffer_updates.begin().get_off() < prev_size)) ||
-     (op.truncate &&
-      (op.truncate->first < prev_size)));
-}
-
 void ECTransaction::generate_transactions(
   WritePlan &plan,
   ErasureCodeInterfaceRef &ecimpl,
diff --git a/src/osd/ECTransaction.h b/src/osd/ECTransaction.h
index 5cb16261a7d4..d889dfbc248a 100644
--- a/src/osd/ECTransaction.h
+++ b/src/osd/ECTransaction.h
@@ -32,10 +32,6 @@ namespace ECTransaction {
     std::map<hobject_t,ECUtil::HashInfoRef> hash_infos;
   };
 
-  bool requires_overwrite(
-    uint64_t prev_size,
-    const PGTransaction::ObjectOperation &op);
-
   template <typename F>
   WritePlan get_write_plan(
     const ECUtil::stripe_info_t &sinfo,