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

wlo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/gobblin.git


The following commit(s) were added to refs/heads/master by this push:
     new 3dffd9c39 [GOBBLIN-2036] Fix type error where string config is forced 
into boolean type (#3915)
3dffd9c39 is described below

commit 3dffd9c3913c1fddb08a5a238f63e5491df12722
Author: William Lo <[email protected]>
AuthorDate: Thu Apr 4 14:31:14 2024 -0400

    [GOBBLIN-2036] Fix type error where string config is forced into boolean 
type (#3915)
---
 .../service/modules/utils/FlowCompilationValidationHelper.java        | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/gobblin-service/src/main/java/org/apache/gobblin/service/modules/utils/FlowCompilationValidationHelper.java
 
b/gobblin-service/src/main/java/org/apache/gobblin/service/modules/utils/FlowCompilationValidationHelper.java
index 38e71acd7..4e9f48e5b 100644
--- 
a/gobblin-service/src/main/java/org/apache/gobblin/service/modules/utils/FlowCompilationValidationHelper.java
+++ 
b/gobblin-service/src/main/java/org/apache/gobblin/service/modules/utils/FlowCompilationValidationHelper.java
@@ -132,8 +132,8 @@ public class FlowCompilationValidationHelper {
    */
   public Optional<Dag<JobExecutionPlan>> 
validateAndHandleConcurrentExecution(Config flowConfig, FlowSpec flowSpec,
       String flowGroup, String flowName, Map<String,String> flowMetadata) 
throws IOException {
-    boolean allowConcurrentExecution = ConfigUtils.getBoolean(flowConfig,
-        ConfigurationKeys.FLOW_ALLOW_CONCURRENT_EXECUTION, 
isFlowConcurrencyEnabled);
+    boolean allowConcurrentExecution = 
Boolean.parseBoolean(ConfigUtils.getString(flowConfig,
+        ConfigurationKeys.FLOW_ALLOW_CONCURRENT_EXECUTION, 
String.valueOf(this.isFlowConcurrencyEnabled)));
 
     Dag<JobExecutionPlan> jobExecutionPlanDag = 
specCompiler.compileFlow(flowSpec);
     if (jobExecutionPlanDag.isEmpty()) {

Reply via email to