adoroszlai commented on code in PR #9646:
URL: https://github.com/apache/ozone/pull/9646#discussion_r2704646319
##########
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/volume/VolumeUsage.java:
##########
@@ -111,6 +111,20 @@ SpaceUsageSource.Fixed realUsage() {
return source.snapshot();
}
+ /**
+ * @return raw filesystem capacity (cached) for the configured volume path.
+ */
+ public long getFsCapacity() {
+ return source.getCapacity();
+ }
+
+ /**
+ * @return raw filesystem available space (cached) for the configured volume
path.
+ */
+ public long getFsAvailable() {
+ return source.getAvailable();
+ }
+
Review Comment:
Thanks @siddhantsangwan for the patch.
Instead of adding these new methods in `VolumeUsage`, I suggest getting the
same values via `realUsage()` in a single call, and passing that to
`getCurrentUsage()`.
[HDDS-14446.patch](https://github.com/user-attachments/files/24716018/HDDS-14446.patch)
--
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]
