This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/spark-kubernetes-operator.git
The following commit(s) were added to refs/heads/main by this push:
new 8daab7a [SPARK-55269] Fix a wrong comment of
`testHandleSentinelResourceReconciliation`
8daab7a is described below
commit 8daab7aa64c14437df00a7f5b6c186047ea845f1
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Thu Jan 29 15:57:19 2026 +0900
[SPARK-55269] Fix a wrong comment of
`testHandleSentinelResourceReconciliation`
### What changes were proposed in this pull request?
This PR aims to fix a wrong comment of
`testHandleSentinelResourceReconciliation` to match with the actual test logic.
### Why are the changes needed?
According to the code, `SENTINEL_RESOURCE_RECONCILIATION_DELAY` is reduced
to `10`, not `0`.
https://github.com/apache/spark-kubernetes-operator/blob/fea548118dc4e7daf192595e20cebd8fd5bc12c3/spark-operator/src/test/java/org/apache/spark/k8s/operator/metrics/healthcheck/SentinelManagerTest.java#L99-L102
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Pass the CIs.
### Was this patch authored or co-authored using generative AI tooling?
Yes (`Opus 4.5` on `Claude Code v2.1.5`)
Closes #474 from dongjoon-hyun/SPARK-55269.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
.../spark/k8s/operator/metrics/healthcheck/SentinelManagerTest.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/spark-operator/src/test/java/org/apache/spark/k8s/operator/metrics/healthcheck/SentinelManagerTest.java
b/spark-operator/src/test/java/org/apache/spark/k8s/operator/metrics/healthcheck/SentinelManagerTest.java
index 26ef1c9..1419f44 100644
---
a/spark-operator/src/test/java/org/apache/spark/k8s/operator/metrics/healthcheck/SentinelManagerTest.java
+++
b/spark-operator/src/test/java/org/apache/spark/k8s/operator/metrics/healthcheck/SentinelManagerTest.java
@@ -96,7 +96,7 @@ class SentinelManagerTest {
@Test
@Order(3)
void testHandleSentinelResourceReconciliation() throws InterruptedException {
- // Reduce the SENTINEL_RESOURCE_RECONCILIATION_DELAY time to 0
+ // Reduce the SENTINEL_RESOURCE_RECONCILIATION_DELAY time to 10
SparkOperatorConfManager.INSTANCE.refresh(
Map.of(
SparkOperatorConf.SENTINEL_RESOURCE_RECONCILIATION_DELAY.getKey(),
"10"));
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
