RexXiong commented on code in PR #3206:
URL: https://github.com/apache/celeborn/pull/3206#discussion_r2035083277
##########
worker/src/main/scala/org/apache/celeborn/service/deploy/worker/storage/DeviceMonitor.scala:
##########
@@ -141,19 +141,34 @@ class LocalDeviceMonitor(
device.notifyObserversOnNonCriticalError(mountPoints,
DiskStatus.IO_HANG)
} else {
device.diskInfos.values().asScala.foreach { diskInfo =>
- if (checkDiskUsage && DeviceMonitor.highDiskUsage(conf,
diskInfo)) {
- logError(s"${diskInfo.mountPoint} high_disk_usage error,
notify observers")
-
device.notifyObserversOnHighDiskUsage(diskInfo.mountPoint)
- } else if (checkReadWrite &&
- DeviceMonitor.readWriteError(conf, diskInfo.dirs.head)) {
- logError(s"${diskInfo.mountPoint} read-write error,
notify observers")
- // We think that if one dir in device has read-write
problem, if possible all
- // dirs in this device have the problem
- device.notifyObserversOnNonCriticalError(
- List(diskInfo.mountPoint),
- DiskStatus.READ_OR_WRITE_FAILURE)
- } else if (nonCriticalErrorSum <=
device.notifyErrorThreshold * 0.5) {
- device.notifyObserversOnHealthy(diskInfo.mountPoint)
+ try {
+ if (checkDiskUsage && DeviceMonitor.highDiskUsage(conf,
diskInfo)) {
+ logError(s"${diskInfo.mountPoint} high_disk_usage
error, notify observers")
+
device.notifyObserversOnHighDiskUsage(diskInfo.mountPoint)
+ } else if (checkReadWrite &&
+ DeviceMonitor.readWriteError(conf,
diskInfo.dirs.head)) {
+ logError(s"${diskInfo.mountPoint} read-write error,
notify observers")
+ // We think that if one dir in device has read-write
problem, if possible all
+ // dirs in this device have the problem
+ device.notifyObserversOnNonCriticalError(
+ List(diskInfo.mountPoint),
+ DiskStatus.READ_OR_WRITE_FAILURE)
+ } else if (nonCriticalErrorSum <=
device.notifyErrorThreshold * 0.5) {
+ device.notifyObserversOnHealthy(diskInfo.mountPoint)
+ }
+ } catch {
+ case e: ExecutionException =>
+ e.getCause match {
+ case fse: FileSystemException =>
+ logError(
+ s"${diskInfo.mountPoint} critical error, notify
observers",
+ fse)
+ device.notifyObserversOnError(
+ List(diskInfo.mountPoint),
+ DiskStatus.CRITICAL_ERROR)
+ case throwable: Throwable =>
+ throw throwable
Review Comment:
nit: Better log the exception.
--
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]
