yandrey321 commented on code in PR #9947:
URL: https://github.com/apache/ozone/pull/9947#discussion_r3192454979
##########
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/volume/HddsVolume.java:
##########
@@ -322,8 +342,20 @@ public VolumeCheckResult checkDbHealth(File dbFile) throws
InterruptedException
throw new InterruptedException("Check of database for volume " + this
+ " interrupted.");
}
- LOG.error("Could not open Volume DB located at {}", dbFile, e);
- getIoTestSlidingWindow().add();
+ // openAsSecondary writes its info LOG into secondaryDir. If that write
+ // fails because the disk is full, RocksDB surfaces the failure as
+ // IOError(NoSpace) (mapped from ENOSPC). That is unrelated to DB
+ // integrity, so don't count it against the sliding window. Any other
+ // status (permission denied, missing path, corruption, generic IO
+ // error) is still treated as a real failure.
+ if (ManagedRocksDB.isNoSpaceFailure(e)) {
Review Comment:
No space issue should be bubbled up, IMHO we should have a check when
metadata volumes are running below 10-15% of the free space
--
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]
