Remove synchronization from FileSystem implementations.

PiperOrigin-RevId: 171905267

1 files changed, 9 insertions, 11 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/vfs/AbstractFileSystem.java b/src/main/java/com/google/devtools/build/lib/vfs/AbstractFileSystem.java
index a03ec02fc5..d1f3a4cd01 100644
--- a/src/main/java/com/google/devtools/build/lib/vfs/AbstractFileSystem.java
+++ b/src/main/java/com/google/devtools/build/lib/vfs/AbstractFileSystem.java
@@ -90,18 +90,16 @@ abstract class AbstractFileSystem extends FileSystem {
 
   @Override
   protected OutputStream getOutputStream(Path path, boolean append) throws IOException {
-    synchronized (path) {
-      try {
-        return createFileOutputStream(path, append);
-      } catch (FileNotFoundException e) {
-        // Why does it throw a *FileNotFoundException* if it can't write?
-        // That does not make any sense! And its in a completely different
-        // format than in other situations, no less!
-        if (e.getMessage().equals(path + ERR_PERMISSION_DENIED)) {
-          throw new FileAccessException(e.getMessage());
-        }
-        throw e;
+    try {
+      return createFileOutputStream(path, append);
+    } catch (FileNotFoundException e) {
+      // Why does it throw a *FileNotFoundException* if it can't write?
+      // That does not make any sense! And its in a completely different
+      // format than in other situations, no less!
+      if (e.getMessage().equals(path + ERR_PERMISSION_DENIED)) {
+        throw new FileAccessException(e.getMessage());
       }
+      throw e;
     }
   }