Skip to content

Commit b93c054

Browse files
committed
Fix code comments based on feedback
1 parent 639e132 commit b93c054

File tree

1 file changed

+9
-7
lines changed
  • lib/src/main/java/com/scalar/admin/kubernetes

1 file changed

+9
-7
lines changed

lib/src/main/java/com/scalar/admin/kubernetes/Pauser.java

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,8 @@ PauserException buildException(
247247
pauserException = unpauseFailedException;
248248
}
249249

250-
// Treat the pause failure as the second priority because the issue might be caused by a configuration mistake.
250+
// Treat the pause failure as the second priority because the issue might be caused by a
251+
// configuration mistake.
251252
if (pauseFailedException != null) {
252253
if (pauserException == null) {
253254
pauserException = pauseFailedException;
@@ -256,8 +257,8 @@ PauserException buildException(
256257
}
257258
}
258259

259-
// Treat the getting target failure as the third priority because this issue might be caused by a temporary glitch, for
260-
// example, network failures.
260+
// Treat the getting target failure as the third priority because this issue might be caused by
261+
// a temporary glitch, for example, network failures.
261262
if (getTargetAfterPauseFailedException != null) {
262263
if (pauserException == null) {
263264
pauserException = getTargetAfterPauseFailedException;
@@ -266,8 +267,9 @@ PauserException buildException(
266267
}
267268
}
268269

269-
// Treat the status checking failure as third priority because this issue might be caused by a temporary glitch,
270-
// for example, targetAfterPause is null.
270+
// Treat the status checking failure as the third priority because this issue might be caused by
271+
// a temporary glitch, for example, getting target information by using the Kubernetes API fails
272+
// after the pause operation.
271273
if (statusCheckFailedException != null) {
272274
if (pauserException == null) {
273275
pauserException = statusCheckFailedException;
@@ -276,8 +278,8 @@ PauserException buildException(
276278
}
277279
}
278280

279-
// Treat the status unmatched issue as third priority because this issue might be caused by temporary
280-
// glitch, for example, pod restarts.
281+
// Treat the status unmatched issue as the third priority because this issue might be caused by
282+
// temporary glitch, for example, pod restarts.
281283
if (statusUnmatchedException != null) {
282284
if (pauserException == null) {
283285
pauserException = statusUnmatchedException;

0 commit comments

Comments
 (0)