Repository: hadoop Updated Branches: refs/heads/HADOOP-15407 1cf38a38d -> 524776625
HADOOP-15715. ITestAzureBlobFileSystemE2E timing out with non-scale timeout of 10 min. Contributed by Da Zhou Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/52477662 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/52477662 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/52477662 Branch: refs/heads/HADOOP-15407 Commit: 524776625d14f6938d89edb923ed05683df819ff Parents: 1cf38a3 Author: Steve Loughran <ste...@apache.org> Authored: Tue Sep 18 11:48:46 2018 +0100 Committer: Steve Loughran <ste...@apache.org> Committed: Tue Sep 18 11:48:46 2018 +0100 ---------------------------------------------------------------------- .../azurebfs/ITestAzureBlobFileSystemE2E.java | 44 +++++++++++--------- .../constants/TestConfigurationKeys.java | 2 +- 2 files changed, 26 insertions(+), 20 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/52477662/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemE2E.java ---------------------------------------------------------------------- diff --git a/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemE2E.java b/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemE2E.java index 6021e5d..2e99491 100644 --- a/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemE2E.java +++ b/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemE2E.java @@ -38,7 +38,6 @@ import static org.junit.Assert.assertArrayEquals; * Test end to end between ABFS client and ABFS server. */ public class ITestAzureBlobFileSystemE2E extends AbstractAbfsIntegrationTest { - private static final Path TEST_FILE = new Path("testfile"); private static final int TEST_BYTE = 100; private static final int TEST_OFFSET = 100; private static final int TEST_DEFAULT_BUFFER_SIZE = 4 * 1024 * 1024; @@ -52,21 +51,16 @@ public class ITestAzureBlobFileSystemE2E extends AbstractAbfsIntegrationTest { @Test public void testWriteOneByteToFile() throws Exception { - final AzureBlobFileSystem fs = getFileSystem(); - - try(FSDataOutputStream stream = fs.create(TEST_FILE)) { - stream.write(TEST_BYTE); - } - - FileStatus fileStatus = fs.getFileStatus(TEST_FILE); - assertEquals(1, fileStatus.getLen()); + final Path testFilePath = new Path(methodName.getMethodName()); + testWriteOneByteToFile(testFilePath); } @Test public void testReadWriteBytesToFile() throws Exception { final AzureBlobFileSystem fs = getFileSystem(); - testWriteOneByteToFile(); - try(FSDataInputStream inputStream = fs.open(TEST_FILE, + final Path testFilePath = new Path(methodName.getMethodName()); + testWriteOneByteToFile(testFilePath); + try(FSDataInputStream inputStream = fs.open(testFilePath, TEST_DEFAULT_BUFFER_SIZE)) { assertEquals(TEST_BYTE, inputStream.read()); } @@ -81,17 +75,17 @@ public class ITestAzureBlobFileSystemE2E extends AbstractAbfsIntegrationTest { final byte[] b = new byte[2 * readBufferSize]; new Random().nextBytes(b); - - try(FSDataOutputStream writeStream = fs.create(TEST_FILE)) { + final Path testFilePath = new Path(methodName.getMethodName()); + try(FSDataOutputStream writeStream = fs.create(testFilePath)) { writeStream.write(b); writeStream.flush(); } - try (FSDataInputStream readStream = fs.open(TEST_FILE)) { + try (FSDataInputStream readStream = fs.open(testFilePath)) { assertEquals(readBufferSize, readStream.read(bytesToRead, 0, readBufferSize)); - try (FSDataOutputStream writeStream = fs.create(TEST_FILE)) { + try (FSDataOutputStream writeStream = fs.create(testFilePath)) { writeStream.write(b); writeStream.flush(); } @@ -104,15 +98,16 @@ public class ITestAzureBlobFileSystemE2E extends AbstractAbfsIntegrationTest { @Test public void testWriteWithBufferOffset() throws Exception { final AzureBlobFileSystem fs = getFileSystem(); + final Path testFilePath = new Path(methodName.getMethodName()); final byte[] b = new byte[1024 * 1000]; new Random().nextBytes(b); - try (FSDataOutputStream stream = fs.create(TEST_FILE)) { + try (FSDataOutputStream stream = fs.create(testFilePath)) { stream.write(b, TEST_OFFSET, b.length - TEST_OFFSET); } final byte[] r = new byte[TEST_DEFAULT_READ_BUFFER_SIZE]; - FSDataInputStream inputStream = fs.open(TEST_FILE, TEST_DEFAULT_BUFFER_SIZE); + FSDataInputStream inputStream = fs.open(testFilePath, TEST_DEFAULT_BUFFER_SIZE); int result = inputStream.read(r); assertNotEquals(-1, result); @@ -124,13 +119,14 @@ public class ITestAzureBlobFileSystemE2E extends AbstractAbfsIntegrationTest { @Test public void testReadWriteHeavyBytesToFileWithSmallerChunks() throws Exception { final AzureBlobFileSystem fs = getFileSystem(); + final Path testFilePath = new Path(methodName.getMethodName()); final byte[] writeBuffer = new byte[5 * 1000 * 1024]; new Random().nextBytes(writeBuffer); - write(TEST_FILE, writeBuffer); + write(testFilePath, writeBuffer); final byte[] readBuffer = new byte[5 * 1000 * 1024]; - FSDataInputStream inputStream = fs.open(TEST_FILE, TEST_DEFAULT_BUFFER_SIZE); + FSDataInputStream inputStream = fs.open(testFilePath, TEST_DEFAULT_BUFFER_SIZE); int offset = 0; while (inputStream.read(readBuffer, offset, TEST_OFFSET) > 0) { offset += TEST_OFFSET; @@ -139,4 +135,14 @@ public class ITestAzureBlobFileSystemE2E extends AbstractAbfsIntegrationTest { assertArrayEquals(readBuffer, writeBuffer); inputStream.close(); } + + private void testWriteOneByteToFile(Path testFilePath) throws Exception { + final AzureBlobFileSystem fs = getFileSystem(); + try(FSDataOutputStream stream = fs.create(testFilePath)) { + stream.write(TEST_BYTE); + } + + FileStatus fileStatus = fs.getFileStatus(testFilePath); + assertEquals(1, fileStatus.getLen()); + } } http://git-wip-us.apache.org/repos/asf/hadoop/blob/52477662/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/constants/TestConfigurationKeys.java ---------------------------------------------------------------------- diff --git a/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/constants/TestConfigurationKeys.java b/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/constants/TestConfigurationKeys.java index 49a0e2e..5565a49 100644 --- a/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/constants/TestConfigurationKeys.java +++ b/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/constants/TestConfigurationKeys.java @@ -35,7 +35,7 @@ public final class TestConfigurationKeys { public static final String TEST_CONFIGURATION_FILE_NAME = "azure-test.xml"; public static final String TEST_CONTAINER_PREFIX = "abfs-testcontainer-"; - public static final int TEST_TIMEOUT = 10 * 60 * 1000; + public static final int TEST_TIMEOUT = 15 * 60 * 1000; private TestConfigurationKeys() {} } --------------------------------------------------------------------- To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-commits-h...@hadoop.apache.org