This is an automated email from the ASF dual-hosted git repository.

jmclean pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new 7808932479 [#7640] add missing validate call to build method in 
SparkJobTemplate (#7657)
7808932479 is described below

commit 7808932479f1b93c9def2570c778b4c7c4b88452
Author: Yunchi Pang <[email protected]>
AuthorDate: Thu Jul 10 17:40:06 2025 -0700

    [#7640] add missing validate call to build method in SparkJobTemplate 
(#7657)
    
    ### What changes were proposed in this pull request?
    
    Add a call of the existing `validation` method before object
    construction in `build`.
    
    ### Why are the changes needed?
    
    `validate` method exists with proper validation logic but is never
    called during object construction.
    Fix: #7640
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Existing tests.
---
 api/src/main/java/org/apache/gravitino/job/SparkJobTemplate.java | 1 +
 1 file changed, 1 insertion(+)

diff --git a/api/src/main/java/org/apache/gravitino/job/SparkJobTemplate.java 
b/api/src/main/java/org/apache/gravitino/job/SparkJobTemplate.java
index f2a6da13a8..6b9d2b677c 100644
--- a/api/src/main/java/org/apache/gravitino/job/SparkJobTemplate.java
+++ b/api/src/main/java/org/apache/gravitino/job/SparkJobTemplate.java
@@ -240,6 +240,7 @@ public class SparkJobTemplate extends JobTemplate {
      */
     @Override
     public SparkJobTemplate build() {
+      validate();
       return new SparkJobTemplate(this);
     }
 

Reply via email to