Skip to content

Commit 92f775c

Browse files
committed
update contract tests and stream leak test
1 parent 5bc0c19 commit 92f775c

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed

hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/ITestS3AFileSystemContract.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,7 @@
3434
import org.apache.hadoop.fs.Path;
3535

3636
import static org.apache.hadoop.fs.contract.ContractTestUtils.skip;
37-
import static org.apache.hadoop.fs.s3a.S3ATestUtils.isCreatePerformanceEnabled;
38-
import static org.apache.hadoop.fs.s3a.S3ATestUtils.setPerformanceFlags;
37+
import static org.apache.hadoop.fs.s3a.S3ATestUtils.*;
3938
import static org.apache.hadoop.test.LambdaTestUtils.intercept;
4039
import static org.junit.Assume.*;
4140
import static org.junit.Assert.*;
@@ -160,4 +159,10 @@ public void testOverwrite() throws IOException {
160159
}
161160
}
162161
}
162+
163+
@Override
164+
public void testOverWriteAndRead() throws Exception {
165+
skipIfAnalyticsAcceleratorEnabled(fs.getConf());
166+
super.testOverWriteAndRead();
167+
}
163168
}

hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/ITestS3AInputStreamLeakage.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434

3535
import static org.apache.hadoop.fs.contract.ContractTestUtils.dataset;
3636
import static org.apache.hadoop.fs.s3a.S3ATestUtils.assume;
37+
import static org.apache.hadoop.fs.s3a.S3ATestUtils.skipIfAnalyticsAcceleratorEnabled;
3738
import static org.apache.hadoop.fs.statistics.IOStatisticAssertions.assertThatStatisticCounter;
3839
import static org.apache.hadoop.fs.statistics.StreamStatisticNames.STREAM_LEAKS;
3940
import static org.apache.hadoop.test.GenericTestUtils.LogCapturer.captureLogs;
@@ -58,7 +59,7 @@ public class ITestS3AInputStreamLeakage extends AbstractS3ATestBase {
5859
@Override
5960
public void setup() throws Exception {
6061
super.setup();
61-
assume("Stream leak detection not avaialable",
62+
assume("Stream leak detection not available",
6263
getFileSystem().hasCapability(STREAM_LEAKS));
6364
}
6465

@@ -89,6 +90,8 @@ public void setup() throws Exception {
8990
@Test
9091
public void testFinalizer() throws Throwable {
9192
Path path = methodPath();
93+
skipIfAnalyticsAcceleratorEnabled(getConfiguration());
94+
9295
final S3AFileSystem fs = getFileSystem();
9396

9497
ContractTestUtils.createFile(fs, path, true, DATASET);

0 commit comments

Comments
 (0)