Skip to content

Commit ccebd3c

Browse files
committed
🔇 remove logs used to debug
Issue: CLDSRV-765
1 parent fee404f commit ccebd3c

File tree

2 files changed

+2
-38
lines changed

2 files changed

+2
-38
lines changed

lib/api/completeMultipartUpload.js

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -539,17 +539,7 @@ function completeMultipartUpload(authInfo, request, log, callback) {
539539
services.batchDeleteObjectMetadata(mpuBucket.getName(),
540540
keysToDelete, log, err => {
541541
if (err) {
542-
log.warn(
543-
'error deleting MPU parts metadata',
544-
{ error: err, method: 'deletePartsMetadata' },
545-
);
546-
console.info({ errorCustom: JSON.stringify(err) });
547-
// Handle specific error cases according to retry strategy
548542
if (err.is?.DeleteConflict) {
549-
log.warn(
550-
'conflict deleting MPU parts metadata, will retry',
551-
{ method: 'deletePartsMetadata' },
552-
);
553543
// DeleteConflict should trigger automatic retry
554544
// Convert to InternalError to make it retryable
555545
const customErr = errorInstances.InternalError.customizeDescription(
@@ -562,22 +552,16 @@ function completeMultipartUpload(authInfo, request, log, callback) {
562552

563553
// For NoSuchKey and other errors, return them as-is
564554
// NoSuchKey is non-retryable, InternalError and others are retryable
565-
log.warn(
566-
'error deleting MPU parts metadata, returning error',
567-
{ method: 'deletePartsMetadata' },
568-
);
569555
return next(err, extraPartLocations,
570556
destinationBucket, aggregateETag, generatedVersionId, droppedMPUSize);
571557
}
572558

573-
log.warn('Successfully deleted MPU parts metadata', { method: 'deletePartsMetadta' });
574559
return next(null, extraPartLocations,
575560
destinationBucket, aggregateETag, generatedVersionId, droppedMPUSize);
576561
});
577562
},
578563
function batchDeleteExtraParts(extraPartLocations, destinationBucket,
579564
aggregateETag, generatedVersionId, droppedMPUSize, next) {
580-
log.warn('deleting extra parts if any', { method: 'batchDeleteExtraParts' });
581565
if (extraPartLocations && extraPartLocations.length > 0) {
582566
return data.batchDelete(extraPartLocations, request.method, null, log, err => {
583567
if (err) {

lib/services.js

Lines changed: 2 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -984,28 +984,8 @@ const services = {
984984
// all at once in production implementation
985985
assert.strictEqual(typeof mpuBucketName, 'string');
986986
async.eachLimit(keysToDelete, 5, (key, callback) => {
987-
metadata.deleteObjectMD(mpuBucketName, key, { overheadField: constants.overheadField }, log, err => {
988-
if (err) {
989-
console.info({ errorInBatch: err });
990-
log.error('error deleting part metadata in batch', {
991-
error: err,
992-
method: 'services.batchDeleteObjectMetadata',
993-
key,
994-
});
995-
}
996-
return callback(err);
997-
});
998-
}, err => {
999-
if (err) {
1000-
console.info({ finalErrorInBatch: err });
1001-
log.error('error in batch (final) deleting part metadata', {
1002-
error: err,
1003-
method: 'services.batchDeleteObjectMetadata',
1004-
});
1005-
}
1006-
1007-
return cb(err);
1008-
});
987+
metadata.deleteObjectMD(mpuBucketName, key, { overheadField: constants.overheadField }, log, callback);
988+
}, cb);
1009989
},
1010990
};
1011991

0 commit comments

Comments
 (0)