swamirishi commented on code in PR #9089:
URL: https://github.com/apache/ozone/pull/9089#discussion_r2409066453
##########
hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/request/OMRequestTestUtils.java:
##########
@@ -706,19 +706,18 @@ public static void addVolumeToDB(String volumeName,
String ownerName,
* @param omMetadataManager
* @throws Exception
*/
- public static void addBucketToDB(String volumeName, String bucketName,
+ public static long addBucketToDB(String volumeName, String bucketName,
OMMetadataManager omMetadataManager) throws Exception {
-
- addBucketToDB(volumeName, bucketName, omMetadataManager,
- BucketLayout.DEFAULT);
+ return addBucketToDB(volumeName, bucketName, omMetadataManager,
+ BucketLayout.DEFAULT).getObjectID();
}
public static OmBucketInfo addBucketToDB(String volumeName,
String bucketName, OMMetadataManager omMetadataManager,
BucketLayout bucketLayout)
throws Exception {
return addBucketToDB(omMetadataManager,
- OmBucketInfo.newBuilder().setVolumeName(volumeName)
+
OmBucketInfo.newBuilder().setVolumeName(volumeName).setObjectID(System.currentTimeMillis())
.setBucketName(bucketName)
.setBucketLayout(bucketLayout)
Review Comment:
done
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/response/snapshot/OMSnapshotMoveDeletedKeysResponse.java:
##########
@@ -156,7 +160,8 @@ private void processReclaimKeys(BatchOperation batchOp,
throws IOException {
for (SnapshotMoveKeyInfos dBKey : reclaimKeysList) {
RepeatedOmKeyInfo omKeyInfos =
- createRepeatedOmKeyInfo(dBKey.getKeyInfosList());
+ createRepeatedOmKeyInfo(dBKey.
+ getKeyInfosList(), bucketId);
Review Comment:
done
##########
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/helpers/RepeatedOmKeyInfo.java:
##########
@@ -41,6 +41,7 @@ public class RepeatedOmKeyInfo implements
CopyObject<RepeatedOmKeyInfo> {
private static final Codec<RepeatedOmKeyInfo> CODEC_FALSE = newCodec(false);
private final List<OmKeyInfo> omKeyInfoList;
+ private final long bucketId;
Review Comment:
done
--
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]
