On second thought, revert 2a8a5c7629984e51b717494a23c0c6651de0b030.

We are only removing ENT when we know for sure that there are other
records for that name, so there is no risk of orphaning NSEC3 chains.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

diff --git a/modules/lmdbbackend/lmdbbackend.cc b/modules/lmdbbackend/lmdbbackend.cc
index 6982323b4be0917a7a22781bbd891ba2b6c5e144..300b6bf2c1180883e78dc1a1477f8a21c437aff8 100644 (file)
--- a/modules/lmdbbackend/lmdbbackend.cc
+++ b/modules/lmdbbackend/lmdbbackend.cc
@@ -2828,11 +2828,6 @@ bool LMDBBackend::updateEmptyNonTerminals(domainid_t domain_id, set<DNSName>& in
       // cout <<" -"<<n<<endl;
       n.makeUsRelative(di.zone);
       txn->txn->del(txn->db->dbi, co(domain_id, n, QType::ENT));
-      // Remove possible orphaned NSEC3 record pair tied to that ENT.
-      auto cursor = txn->txn->getCursor(txn->db->dbi);
-      if (hasOrphanedNSEC3Record(cursor, domain_id, n)) {
-        deleteNSEC3RecordPair(txn, domain_id, n);
-      }
     }
   }
   if (needCommit)