Resolve comments

core/src/main/java/com/datastrato/gravitino/storage/kv/KvGarbageCollector.java

@@ -193,14 +193,15 @@ private void collectAndRemoveOldVersionData() throws IOException {
           removeAllVersionsOfKey(rawKey, key, false);
 
           LogHelper logHelper = decodeKey(key, transactionId);
+          kvBackend.delete(rawKey);
           LOG.info(
-              "Physically delete key that has marked deleted: name identifier: '{}', entity type: '{}', createTime: '{}({})', key: '{}'",
+              "Physically delete key that has marked deleted: name identifier: '{}', entity type: '{}',"
+                  + " createTime: '{}({})', key: '{}'",
               logHelper.identifier,
               logHelper.type,
               logHelper.createTimeAsString,
               logHelper.createTimeInMs,
               Bytes.wrap(key));
-          kvBackend.delete(rawKey);
           keysDeletedCount++;
           continue;
         }
@@ -225,8 +226,10 @@ private void collectAndRemoveOldVersionData() throws IOException {
           LogHelper logHelper = decodeKey(key, transactionId);
           byte[] newVersionKey = newVersionOfKey.get(0).getKey();
           LogHelper newVersionLogHelper = decodeKey(newVersionKey);
+          kvBackend.delete(rawKey);
           LOG.info(
-              "Physically delete key that has newer version: name identifier: '{}', entity type: '{}', createTime: '{}({})', newVersion createTime: '{}({})',"
+              "Physically delete key that has newer version: name identifier: '{}', entity type: '{}',"
+                  + " createTime: '{}({})', newVersion createTime: '{}({})',"
                   + " key: '{}', newVersion key: '{}'",
               logHelper.identifier,
               logHelper.type,
@@ -236,21 +239,20 @@ private void collectAndRemoveOldVersionData() throws IOException {
               newVersionLogHelper.createTimeInMs,
               Bytes.wrap(rawKey),
               Bytes.wrap(newVersionKey));
-          kvBackend.delete(rawKey);
           keysDeletedCount++;
         }
       }
 
       // All keys in this transaction have been deleted, we can remove the commit mark.
       if (keysDeletedCount == keysInTheTransaction.size()) {
+        kvBackend.delete(kv.getKey());
         long timestamp = getTransactionId(transactionId) >> 18;
         LOG.info(
             "Physically delete commit mark: {}, createTime: '{}({})', key: '{}'",
             Bytes.wrap(kv.getKey()),
             DateFormatUtils.format(timestamp, TIME_STAMP_FORMAT),
             timestamp,
             Bytes.wrap(kv.getKey()));
-        kvBackend.delete(kv.getKey());
       }
     }
 
@@ -283,7 +285,8 @@ private void removeAllVersionsOfKey(byte[] rawKey, byte[] key, boolean includeSt
 
       LogHelper logHelper = decodeKey(kv.getKey());
       LOG.info(
-          "Physically delete key that has marked deleted: name identifier: '{}', entity type: '{}', createTime: '{}({})', key: '{}'",
+          "Physically delete key that has marked deleted: name identifier: '{}', entity type: '{}',"
+              + " createTime: '{}({})', key: '{}'",
           logHelper.identifier,
           logHelper.type,
           logHelper.createTimeAsString,
@@ -308,14 +311,14 @@ private void removeAllVersionsOfKey(byte[] rawKey, byte[] key, boolean includeSt
 
       // Try to delete the commit mark.
       if (allDropped) {
+        kvBackend.delete(transactionKey);
         long timestamp = TransactionalKvBackendImpl.getTransactionId(transactionId) >> 18;
         LOG.info(
             "Physically delete commit mark: {}, createTime: '{}({})', key: '{}'",
             Bytes.wrap(kv.getKey()),
             DateFormatUtils.format(timestamp, TIME_STAMP_FORMAT),
             timestamp,
             Bytes.wrap(kv.getKey()));
-        kvBackend.delete(transactionKey);
       }
     }
   }