tanvipenumudy commented on code in PR #5745:
URL: https://github.com/apache/ozone/pull/5745#discussion_r1551284786
##########
hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/response/bucket/TestOMBucketDeleteResponse.java:
##########
@@ -100,7 +102,7 @@ public void testAddToDBBatch() throws Exception {
omMetadataManager.getStore().commitBatchOperation(batchOperation);
assertNull(omMetadataManager.getBucketTable().get(
- omMetadataManager.getBucketKey(volumeName, bucketName)));
+ omMetadataManager.getBucketKey(volumeName, bucketName)));
Review Comment:
Same as the previous comments.
##########
hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/ratis/TestOzoneManagerDoubleBufferWithDummyResponse.java:
##########
@@ -163,14 +164,14 @@ private static class OMDummyCreateBucketResponse extends
OMClientResponse {
private final OmBucketInfo omBucketInfo;
OMDummyCreateBucketResponse(OmBucketInfo omBucketInfo,
- OMResponse omResponse) {
+ OMResponse omResponse) {
Review Comment:
Minor nit: Let's stick to the original indent.
##########
hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/ratis/TestOzoneManagerDoubleBufferWithDummyResponse.java:
##########
@@ -163,14 +164,14 @@ private static class OMDummyCreateBucketResponse extends
OMClientResponse {
private final OmBucketInfo omBucketInfo;
OMDummyCreateBucketResponse(OmBucketInfo omBucketInfo,
- OMResponse omResponse) {
+ OMResponse omResponse) {
super(omResponse);
this.omBucketInfo = omBucketInfo;
}
@Override
public void addToDBBatch(OMMetadataManager omMetadataManager,
- BatchOperation batchOperation) throws IOException {
+ BatchOperation batchOperation) throws IOException
{
Review Comment:
Minor nit: Here as well, we can stick with the original indent.
--
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]
