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 6339d91  [SPARK-53713] Remove `org.jetbrains.annotations` test code 
dependency from `spark-operator`
6339d91 is described below

commit 6339d912bbad1081eb6ccc0d27dfa7f498dcb988
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Thu Sep 25 00:02:36 2025 -0700

    [SPARK-53713] Remove `org.jetbrains.annotations` test code dependency from 
`spark-operator`
    
    ### What changes were proposed in this pull request?
    
    This PR aims to remove `org.jetbrains.annotations` test code dependency 
from `spark-operator`.
    
    ### Why are the changes needed?
    
    To simply `spark-operator` dependency.
    
    **BEFORE**
    
    ```
    $ git grep org.jetbrains | wc -l
           1
    ```
    
    **AFTER**
    
    ```
    $ git grep org.jetbrains | wc -l
           0
    ```
    
    ### Does this PR introduce _any_ user-facing change?
    
    No behavior change. This annotation was used only for the test code.
    
    ### How was this patch tested?
    
    Pass the CIs.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #345 from dongjoon-hyun/SPARK-53713.
    
    Authored-by: Dongjoon Hyun <[email protected]>
    Signed-off-by: Dongjoon Hyun <[email protected]>
---
 .../java/org/apache/spark/k8s/operator/utils/StatusRecorderTest.java    | 2 --
 1 file changed, 2 deletions(-)

diff --git 
a/spark-operator/src/test/java/org/apache/spark/k8s/operator/utils/StatusRecorderTest.java
 
b/spark-operator/src/test/java/org/apache/spark/k8s/operator/utils/StatusRecorderTest.java
index 56bda41..7de0633 100644
--- 
a/spark-operator/src/test/java/org/apache/spark/k8s/operator/utils/StatusRecorderTest.java
+++ 
b/spark-operator/src/test/java/org/apache/spark/k8s/operator/utils/StatusRecorderTest.java
@@ -33,7 +33,6 @@ import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import io.fabric8.kubernetes.client.KubernetesClient;
 import io.fabric8.kubernetes.client.server.mock.EnableKubernetesMockClient;
 import io.fabric8.kubernetes.client.server.mock.KubernetesMockServer;
-import org.jetbrains.annotations.NotNull;
 import org.junit.jupiter.api.Test;
 
 import org.apache.spark.k8s.operator.SparkApplication;
@@ -86,7 +85,6 @@ class StatusRecorderTest {
             any());
   }
 
-  @NotNull
   private static SparkApplication getSparkApplication(String resourceVersion) {
     var updated = TestUtils.createMockApp(DEFAULT_NS);
     updated.getMetadata().setResourceVersion(resourceVersion);


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to