HBASE-20507 Do not need to call recoverLease on the broken file when we fail to create a wal writer
This commit is contained in:
parent
4cb444e77b
commit
09ca619389
|
@ -22,6 +22,7 @@ import static org.junit.Assert.assertEquals;
|
||||||
import static org.junit.Assert.assertThat;
|
import static org.junit.Assert.assertThat;
|
||||||
import static org.junit.Assert.fail;
|
import static org.junit.Assert.fail;
|
||||||
|
|
||||||
|
import java.io.FileNotFoundException;
|
||||||
import java.io.IOException;
|
import java.io.IOException;
|
||||||
import org.apache.hadoop.fs.FSDataInputStream;
|
import org.apache.hadoop.fs.FSDataInputStream;
|
||||||
import org.apache.hadoop.fs.FSDataOutputStream;
|
import org.apache.hadoop.fs.FSDataOutputStream;
|
||||||
|
@ -95,6 +96,8 @@ public class TestOverwriteFileUnderConstruction {
|
||||||
out1.close();
|
out1.close();
|
||||||
// a successful close is also OK for us so no assertion here, we just need to confirm that the
|
// 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.
|
// data in the file are correct.
|
||||||
|
} catch (FileNotFoundException fnfe) {
|
||||||
|
// hadoop3 throws one of these.
|
||||||
} catch (RemoteException e) {
|
} catch (RemoteException e) {
|
||||||
// expected
|
// expected
|
||||||
assertThat(e.unwrapRemoteException(), instanceOf(LeaseExpiredException.class));
|
assertThat(e.unwrapRemoteException(), instanceOf(LeaseExpiredException.class));
|
||||||
|
|
Loading…
Reference in New Issue