fix(metadata): prevent duplicate release for constant instance

metadata/src/main/java/org/apache/kafka/image/RegistryRef.java

@@ -54,10 +54,39 @@ public RegistryRef next() {
     }
 
     @Override
-    protected void deallocate() {
+    public ReferenceCounted retain(int increment) {
         if (this == NOOP) {
-            return;
+            throw new UnsupportedOperationException("retain is not supported for NOOP");
         }
+        return super.retain(increment);
+    }
+
+    @Override
+    public ReferenceCounted retain() {
+        if (this == NOOP) {
+            throw new UnsupportedOperationException("retain is not supported for NOOP");
+        }
+        return super.retain();
+    }
+
+    @Override
+    public boolean release() {
+        if (this == NOOP) {
+            throw new UnsupportedOperationException("release is not supported for NOOP");
+        }
+        return super.release();
+    }
+
+    @Override
+    public boolean release(int decrement) {
+        if (this == NOOP) {
+            throw new UnsupportedOperationException("release is not supported for NOOP");
+        }
+        return super.release(decrement);
+    }
+
+    @Override
+    protected void deallocate() {
         inLock(() -> {
             if (liveEpochs.isEmpty()) {
                 throw new IllegalStateException("liveEpochs is empty");

metadata/src/main/java/org/apache/kafka/image/S3ObjectsImage.java

@@ -139,6 +139,9 @@ RegistryRef registryRef() {
 
     @Override
     protected void deallocate() {
+        if (registryRef == RegistryRef.NOOP) {
+            return;
+        }
         registryRef.release();
     }
 

metadata/src/main/java/org/apache/kafka/image/S3StreamsMetadataImage.java

@@ -408,6 +408,9 @@ public String toString() {
 
     @Override
     protected void deallocate() {
+        if (registryRef == RegistryRef.NOOP) {
+            return;
+        }
         registryRef.release();
     }