diff --git a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestKeyPurging.java b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestKeyPurging.java index 37d22b1720d9..96da1ce901bd 100644 --- a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestKeyPurging.java +++ b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestKeyPurging.java @@ -28,19 +28,16 @@ import java.util.List; import java.util.UUID; import java.util.concurrent.TimeUnit; -import org.apache.hadoop.hdds.client.ReplicationFactor; -import org.apache.hadoop.hdds.client.ReplicationType; import org.apache.hadoop.hdds.conf.OzoneConfiguration; import org.apache.hadoop.hdds.utils.IOUtils; import org.apache.hadoop.ozone.MiniOzoneCluster; +import org.apache.hadoop.ozone.TestDataUtil; import org.apache.hadoop.ozone.client.ObjectStore; import org.apache.hadoop.ozone.client.OzoneBucket; import org.apache.hadoop.ozone.client.OzoneClient; import org.apache.hadoop.ozone.client.OzoneClientFactory; import org.apache.hadoop.ozone.client.OzoneVolume; -import org.apache.hadoop.ozone.client.io.OzoneOutputStream; import org.apache.hadoop.ozone.container.ContainerTestHelper; -import org.apache.hadoop.ozone.container.TestHelper; import org.apache.hadoop.ozone.om.service.KeyDeletingService; import org.apache.ozone.test.GenericTestUtils; import org.junit.jupiter.api.AfterEach; @@ -108,11 +105,7 @@ public void testKeysPurgingByKeyDeletingService() throws Exception { for (int i = 1; i <= NUM_KEYS; i++) { String keyName = keyBase + "-" + i; keys.add(keyName); - OzoneOutputStream keyStream = TestHelper.createKey( - keyName, ReplicationType.RATIS, ReplicationFactor.ONE, - KEY_SIZE, store, volumeName, bucketName); - keyStream.write(data); - keyStream.close(); + TestDataUtil.createKey(bucket, keyName, data); } // Delete created keys diff --git a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOzoneManagerPrepare.java b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOzoneManagerPrepare.java index cd2fc4af2949..edc9b569b2a5 100644 --- a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOzoneManagerPrepare.java +++ b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOzoneManagerPrepare.java @@ -39,15 +39,12 @@ import java.util.concurrent.ExecutorService; import java.util.concurrent.Executors; import java.util.concurrent.Future; -import org.apache.hadoop.hdds.client.ReplicationFactor; -import org.apache.hadoop.hdds.client.ReplicationType; import org.apache.hadoop.ozone.MiniOzoneHAClusterImpl; +import org.apache.hadoop.ozone.TestDataUtil; import org.apache.hadoop.ozone.client.ObjectStore; import org.apache.hadoop.ozone.client.OzoneVolume; -import org.apache.hadoop.ozone.client.io.OzoneOutputStream; import org.apache.hadoop.ozone.client.protocol.ClientProtocol; import org.apache.hadoop.ozone.container.ContainerTestHelper; -import org.apache.hadoop.ozone.container.TestHelper; import org.apache.hadoop.ozone.om.exceptions.OMException; import org.apache.hadoop.ozone.om.helpers.OmKeyInfo; import org.apache.hadoop.ozone.protocol.proto.OzoneManagerProtocolProtos.PrepareStatusResponse.PrepareStatus; @@ -391,11 +388,8 @@ private void writeTestData(String volumeName, String keyString = UUID.randomUUID().toString(); byte[] data = ContainerTestHelper.getFixedLengthString( keyString, 100).getBytes(UTF_8); - OzoneOutputStream keyStream = TestHelper.createKey( - keyName, ReplicationType.RATIS, ReplicationFactor.ONE, - 100, store, volumeName, bucketName); - keyStream.write(data); - keyStream.close(); + TestDataUtil.createKey(store.getVolume(volumeName). + getBucket(bucketName), keyName, data); } private void assertKeysWritten(String volumeName, diff --git a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/recon/TestReconWithOzoneManagerFSO.java b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/recon/TestReconWithOzoneManagerFSO.java index 04c59e6f087a..41c8766c9f7d 100644 --- a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/recon/TestReconWithOzoneManagerFSO.java +++ b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/recon/TestReconWithOzoneManagerFSO.java @@ -24,19 +24,17 @@ import java.util.UUID; import javax.ws.rs.core.Response; -import org.apache.hadoop.hdds.client.ReplicationFactor; -import org.apache.hadoop.hdds.client.ReplicationType; import org.apache.hadoop.hdds.conf.OzoneConfiguration; import org.apache.hadoop.hdds.protocol.proto.HddsProtos; import org.apache.hadoop.hdds.scm.server.OzoneStorageContainerManager; import org.apache.hadoop.hdds.utils.IOUtils; import org.apache.hadoop.ozone.MiniOzoneCluster; +import org.apache.hadoop.ozone.TestDataUtil; import org.apache.hadoop.ozone.client.ObjectStore; +import org.apache.hadoop.ozone.client.OzoneBucket; import org.apache.hadoop.ozone.client.OzoneClient; import org.apache.hadoop.ozone.client.OzoneVolume; -import org.apache.hadoop.ozone.client.io.OzoneOutputStream; import org.apache.hadoop.ozone.container.ContainerTestHelper; -import org.apache.hadoop.ozone.container.TestHelper; import org.apache.hadoop.ozone.om.OMConfigKeys; import org.apache.hadoop.ozone.recon.api.NSSummaryEndpoint; import org.apache.hadoop.ozone.recon.api.types.EntityType; @@ -67,7 +65,7 @@ public static void init() throws Exception { OMConfigKeys.OZONE_BUCKET_LAYOUT_FILE_SYSTEM_OPTIMIZED); cluster = MiniOzoneCluster.newBuilder(conf) - .setNumDatanodes(1) + .setNumDatanodes(3) .includeRecon(true) .build(); cluster.waitForClusterToBeReady(); @@ -85,26 +83,16 @@ public static void shutdown() { } } - private void writeTestData(String volumeName, - String bucketName, - String keyName) throws Exception { - - String keyString = UUID.randomUUID().toString(); - byte[] data = ContainerTestHelper.getFixedLengthString( - keyString, 100).getBytes(UTF_8); - OzoneOutputStream keyStream = TestHelper.createKey( - keyName, ReplicationType.RATIS, ReplicationFactor.ONE, - 100, store, volumeName, bucketName); - keyStream.write(data); - keyStream.close(); - } - private void writeKeys(String vol, String bucket, String key) throws Exception { store.createVolume(vol); OzoneVolume volume = store.getVolume(vol); volume.createBucket(bucket); - writeTestData(vol, bucket, key); + OzoneBucket ozoneBucket = volume.getBucket(bucket); + String keyString = UUID.randomUUID().toString(); + byte[] data = ContainerTestHelper.getFixedLengthString( + keyString, 100).getBytes(UTF_8); + TestDataUtil.createKey(ozoneBucket, key, data); } @Test