Core: Remove unnecessary class-level synchronized in ManifestFiles (#10544)

findepi · web-flow · commit cb6540c8c989 · 2024-07-18T19:30:01.000-06:00
diff --git a/core/src/main/java/org/apache/iceberg/ManifestFiles.java b/core/src/main/java/org/apache/iceberg/ManifestFiles.java
@@ -81,7 +81,7 @@ static ContentCache contentCache(FileIO io) {
   }
 
   /** Drop manifest file cache object for a FileIO if exists. */
-  public static synchronized void dropCache(FileIO fileIO) {
+  public static void dropCache(FileIO fileIO) {
     CONTENT_CACHES.invalidate(fileIO);
     CONTENT_CACHES.cleanUp();
   }

Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ static ContentCache contentCache(FileIO io) {`
`81`	`81`	`}`
`82`	`82`
`83`	`83`	`/** Drop manifest file cache object for a FileIO if exists. */`
`84`		`- public static synchronized void dropCache(FileIO fileIO) {`
	`84`	`+ public static void dropCache(FileIO fileIO) {`
`85`	`85`	`CONTENT_CACHES.invalidate(fileIO);`
`86`	`86`	`CONTENT_CACHES.cleanUp();`
`87`	`87`	`}`