sodonnel commented on a change in pull request #2777:
URL: https://github.com/apache/ozone/pull/2777#discussion_r740149699
##########
File path:
hadoop-ozone/client/src/test/java/org/apache/hadoop/ozone/client/TestOzoneECClient.java
##########
@@ -75,21 +69,19 @@
private byte[][] inputChunks = new byte[dataBlocks][chunkSize];
private final XceiverClientFactory factoryStub =
new MockXceiverClientFactory();
- private MockOmTransport transportStub = null;
- private ECSchema schema = new ECSchema("rs", dataBlocks, parityBlocks);
- private ErasureCodecOptions options = new ErasureCodecOptions(schema);
private OzoneConfiguration conf = new OzoneConfiguration();
- private RSErasureCodec codec = new RSErasureCodec(conf, options);
- private final RawErasureEncoder encoder = CodecUtil.createRawEncoder(conf,
- SystemErasureCodingPolicies.getPolicies().get(1).getCodecName(),
- codec.getCoderOptions());
+ private MockOmTransport transportStub = new MockOmTransport(
+ new MultiNodePipelineBlockAllocator(conf, dataBlocks + parityBlocks));
+ private final RawErasureEncoder encoder =
+ new RSRawErasureCoderFactory().createEncoder(
+ new ECReplicationConfig(dataBlocks, parityBlocks));
@Before
public void init() throws IOException {
conf.setStorageSize(OzoneConfigKeys.OZONE_SCM_BLOCK_SIZE, 2,
StorageUnit.KB);
- transportStub = new MockOmTransport(
- new MultiNodePipelineBlockAllocator(conf, dataBlocks + parityBlocks));
+ //transportStub = new MockOmTransport(
+ // new MultiNodePipelineBlockAllocator(conf, dataBlocks +
parityBlocks));
Review comment:
Should we remove the commented lines here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
