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);
}