Re-added old versions of onCached and onRemoval to ShardRequestCache

Signed-off-by: Peter Alfonsi <[email protected]>

server/src/main/java/org/opensearch/index/cache/request/ShardRequestCache.java

@@ -61,6 +61,7 @@ public void onMiss() {
         missCount.inc();
     }
 
+    // Functions used to increment size by passing in the size directly, Used now, as we use ICacheKey<Key> in the IndicesRequestCache..
     public void onCached(long keyRamBytesUsed, BytesReference value) {
         totalMetric.inc(keyRamBytesUsed + value.ramBytesUsed());
     }
@@ -75,4 +76,22 @@ public void onRemoval(long keyRamBytesUsed, BytesReference value, boolean evicte
         }
         totalMetric.dec(dec);
     }
+
+    // Old functions which increment size by passing in an Accountable. Functional but no longer used.
+    public void onCached(Accountable key, BytesReference value) {
+        totalMetric.inc(key.ramBytesUsed() + value.ramBytesUsed());
+    }
+
+    public void onRemoval(Accountable key, BytesReference value, boolean evicted) {
+        if (evicted) {
+            evictionsMetric.inc();
+        }
+        long dec = 0;
+        if (key != null) {
+            dec += key.ramBytesUsed();
+        }
+        if (value != null) {
+            dec += value.ramBytesUsed();
+        }
+    }
 }