Repository: hbase Updated Branches: refs/heads/branch-2.0 1886e7843 -> d1c729b5c
HBASE-20507 Do not need to call recoverLease on the broken file when we fail to create a wal writer Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/d1c729b5 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/d1c729b5 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/d1c729b5 Branch: refs/heads/branch-2.0 Commit: d1c729b5c4a574bf258fa6e9d1f0c60682e7b966 Parents: 1886e78 Author: Michael Stack <st...@apache.org> Authored: Thu May 3 12:26:12 2018 -0700 Committer: Michael Stack <st...@apache.org> Committed: Thu May 3 12:26:12 2018 -0700 ---------------------------------------------------------------------- .../hbase/io/asyncfs/TestOverwriteFileUnderConstruction.java | 3 +++ 1 file changed, 3 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/d1c729b5/hbase-server/src/test/java/org/apache/hadoop/hbase/io/asyncfs/TestOverwriteFileUnderConstruction.java ---------------------------------------------------------------------- diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/io/asyncfs/TestOverwriteFileUnderConstruction.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/io/asyncfs/TestOverwriteFileUnderConstruction.java index 7f75fdb..aaeed15 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/io/asyncfs/TestOverwriteFileUnderConstruction.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/io/asyncfs/TestOverwriteFileUnderConstruction.java @@ -22,6 +22,7 @@ import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertThat; import static org.junit.Assert.fail; +import java.io.FileNotFoundException; import java.io.IOException; import org.apache.hadoop.fs.FSDataInputStream; import org.apache.hadoop.fs.FSDataOutputStream; @@ -95,6 +96,8 @@ public class TestOverwriteFileUnderConstruction { out1.close(); // a successful close is also OK for us so no assertion here, we just need to confirm that the // data in the file are correct. + } catch (FileNotFoundException fnfe) { + // hadoop3 throws one of these. } catch (RemoteException e) { // expected assertThat(e.unwrapRemoteException(), instanceOf(LeaseExpiredException.class));