Copilot commented on code in PR #9116:
URL: https://github.com/apache/seatunnel/pull/9116#discussion_r2055126884
##########
seatunnel-engine/seatunnel-engine-core/src/main/java/org/apache/seatunnel/engine/core/parse/MultipleTableJobConfigParser.java:
##########
@@ -159,6 +160,24 @@ public MultipleTableJobConfigParser(
this.isStartWithSavePoint = isStartWithSavePoint;
this.seaTunnelJobConfig =
ConfigBuilder.of(Paths.get(jobDefineFilePath), variables);
this.envOptions =
ReadonlyConfig.fromConfig(seaTunnelJobConfig.getConfig("env"));
+
+ String refPath =
envOptions.getOptional(EnvCommonOptions.REF_PATH).orElse(null);
Review Comment:
The handling of the ref_config_path is inconsistent between constructors;
one passes variables to ConfigBuilder while the other uses null. Consider
standardizing the parameter usage to ensure consistent behavior across
instances.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
