[fix][broker] Support null mark delete properties in NonDurableCursorImpl.internalAsyncMarkDelete

Author: oneby-wang

managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/NonDurableCursorImpl.java

@@ -103,11 +103,13 @@ void recover(final VoidCallback callback) {
     protected void internalAsyncMarkDelete(final Position newPosition, Map<String, Long> properties,
             final MarkDeleteCallback callback, final Object ctx, Runnable alignAcknowledgeStatusAfterPersisted) {
         // Bypass persistence of mark-delete position and individually deleted messages info
-
-        MarkDeleteEntry mdEntry = new MarkDeleteEntry(newPosition, properties, callback, ctx,
-                alignAcknowledgeStatusAfterPersisted);
+        MarkDeleteEntry mdEntry;
         lock.writeLock().lock();
         try {
+            // use given properties or when missing, use the properties from the previous field value
+            Map<String, Long> propertiesToUse = properties != null ? properties : getProperties();
+            mdEntry = new MarkDeleteEntry(newPosition, propertiesToUse, callback, ctx,
+                    alignAcknowledgeStatusAfterPersisted);
             lastMarkDeleteEntry = mdEntry;
             mdEntry.alignAcknowledgeStatus();
         } finally {