You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2018/05/03 19:27:03 UTC
[2/2] hbase git commit: HBASE-20507 Do not need to call recoverLease
on the broken file when we fail to create a wal writer
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/e92580e2
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/e92580e2
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/e92580e2
Branch: refs/heads/branch-2
Commit: e92580e2012fdc7b3741bc73af630efe4b7bd82f
Parents: f89c5b0
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:54 2018 -0700
----------------------------------------------------------------------
.../hbase/io/asyncfs/TestOverwriteFileUnderConstruction.java | 3 +++
1 file changed, 3 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/e92580e2/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));