alexr17 commented on code in PR #14022:
URL: https://github.com/apache/hudi/pull/14022#discussion_r2393001602
##########
hudi-aws/src/main/java/org/apache/hudi/aws/transaction/lock/S3StorageLockClient.java:
##########
@@ -188,14 +188,14 @@ private StorageLockFile
createOrUpdateLockFileInternal(StorageLockData lockData,
private LockUpsertResult handleUpsertS3Exception(S3Exception e) {
int status = e.statusCode();
if (status == PRECONDITION_FAILURE_ERROR_CODE) {
- logger.warn("OwnerId: {}, Lockfile modified by another process: {}",
ownerId, lockFilePath);
+ logger.error("OwnerId: {}, Lockfile modified by another process: {}",
ownerId, lockFilePath);
Review Comment:
this can be info
##########
hudi-aws/src/main/java/org/apache/hudi/aws/transaction/lock/S3StorageLockClient.java:
##########
@@ -188,14 +188,14 @@ private StorageLockFile
createOrUpdateLockFileInternal(StorageLockData lockData,
private LockUpsertResult handleUpsertS3Exception(S3Exception e) {
int status = e.statusCode();
if (status == PRECONDITION_FAILURE_ERROR_CODE) {
- logger.warn("OwnerId: {}, Lockfile modified by another process: {}",
ownerId, lockFilePath);
+ logger.error("OwnerId: {}, Lockfile modified by another process: {}",
ownerId, lockFilePath);
return LockUpsertResult.ACQUIRED_BY_OTHERS;
} else if (status == CONDITIONAL_REQUEST_CONFLICT_ERROR_CODE) {
- logger.warn("OwnerId: {}, Retriable conditional request conflict error:
{}", ownerId, lockFilePath);
+ logger.error("OwnerId: {}, Retriable conditional request conflict error:
{}", ownerId, lockFilePath);
Review Comment:
info
--
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]
