Skip to content

Commit 2547220

Browse files
committed
♻️ don't fetch bucket data all the time
Issue: CLDSRV-632
1 parent dfc7d0d commit 2547220

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

lib/routes/routeBackbeat.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ const { versioningPreprocessing }
4545
const {promisify} = require('util');
4646

4747
const versioningPreprocessingPromised = promisify(versioningPreprocessing);
48+
metadata.getObjectMDPromised = promisify(metadata.getObjectMD);
4849
metadata.getBucketAndObjectMDPromised = promisify(metadata.getBucketAndObjectMD);
4950

5051
const { CURRENT_TYPE, NON_CURRENT_TYPE, ORPHAN_DM_TYPE } = constants.lifecycleListing;
@@ -664,16 +665,14 @@ function putMetadata(request, response, bucketInfo, objMd, log, callback) {
664665
},
665666
async () => {
666667
if (versioning && !objMd) {
667-
const masterObjectAndBucket =
668-
await metadata.getBucketAndObjectMDPromised(bucketName, objectKey, {}, log);
668+
const masterMD = await metadata.getObjectMD(bucketName, objectKey, {}, log);
669669

670-
if (!masterObjectAndBucket.obj) {
670+
if (!masterMD) {
671671
return;
672672
}
673673

674-
const masterObject = JSON.parse(masterObjectAndBucket.obj);
675674
const versioningPreprocessingResult =
676-
await versioningPreprocessingPromised(bucketName, bucketInfo, objectKey, masterObject, log);
675+
await versioningPreprocessingPromised(bucketName, bucketInfo, objectKey, masterMD, log);
677676

678677
if (versioningPreprocessingResult?.nullVersionId) {
679678
omVal.nullVersionId = versioningPreprocessingResult.nullVersionId;

0 commit comments

Comments
 (0)