HDFS-9294. DFSClient deadlock when close file and failed to renew lease. Contributed by Brahma Reddy Battula

hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java

@@ -708,13 +708,17 @@ protected synchronized void start() {
    * Aborts this output stream and releases any system
    * resources associated with this stream.
    */
-  synchronized void abort() throws IOException {
+  void abort() throws IOException {
+    synchronized (this) {
       if (isClosed()) {
         return;
       }
-    getStreamer().getLastException().set(new IOException("Lease timeout of "
+      getStreamer().getLastException().set(
-        + (dfsClient.getConf().getHdfsTimeout() / 1000) + " seconds expired."));
+          new IOException("Lease timeout of "
+              + (dfsClient.getConf().getHdfsTimeout() / 1000)
+              + " seconds expired."));
       closeThreads(true);
+    }
     dfsClient.endFileLease(fileId);
   }
 
@@ -747,12 +751,15 @@ protected void closeThreads(boolean force) throws IOException {
    * resources associated with this stream.
    */
   @Override
-  public synchronized void close() throws IOException {
+  public void close() throws IOException {
-    try (TraceScope ignored =
+    synchronized (this) {
-             dfsClient.newPathTraceScope("DFSOutputStream#close", src)) {
+      try (TraceScope ignored = dfsClient.newPathTraceScope(
+          "DFSOutputStream#close", src)) {
         closeImpl();
       }
     }
+    dfsClient.endFileLease(fileId);
+  }
 
   protected synchronized void closeImpl() throws IOException {
     if (isClosed()) {
@@ -779,7 +786,6 @@ protected synchronized void closeImpl() throws IOException {
                dfsClient.getTracer().newScope("completeFile")) {
         completeFile(lastBlock);
       }
-      dfsClient.endFileLease(fileId);
     } catch (ClosedChannelException ignored) {
     } finally {
       setClosed();

hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSStripedOutputStream.java

@@ -757,16 +757,19 @@ protected synchronized void start() {
   }
 
   @Override
-  synchronized void abort() throws IOException {
+  void abort() throws IOException {
+    synchronized (this) {
       if (isClosed()) {
         return;
       }
       for (StripedDataStreamer streamer : streamers) {
-      streamer.getLastException().set(new IOException("Lease timeout of "
+        streamer.getLastException().set(
-          + (dfsClient.getConf().getHdfsTimeout()/1000) +
+            new IOException("Lease timeout of "
-          " seconds expired."));
+                + (dfsClient.getConf().getHdfsTimeout() / 1000)
+                + " seconds expired."));
       }
       closeThreads(true);
+    }
     dfsClient.endFileLease(fileId);
   }
 
@@ -954,7 +957,6 @@ protected synchronized void closeImpl() throws IOException {
                dfsClient.getTracer().newScope("completeFile")) {
         completeFile(currentBlockGroup);
       }
-      dfsClient.endFileLease(fileId);
     } catch (ClosedChannelException ignored) {
     } finally {
       setClosed();

hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt

@@ -2535,6 +2535,9 @@ Release 2.7.2 - UNRELEASED
     HDFS-9426. Rollingupgrade finalization is not backward compatible
     (Kihwal Lee via vinayakumarb)
 
+    HDFS-9294. DFSClient deadlock when close file and failed to renew lease.
+    (Brahma Reddy Battula via szetszwo)
+
 Release 2.7.1 - 2015-07-06
 
   INCOMPATIBLE CHANGES