You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@gobblin.apache.org by su...@apache.org on 2021/02/17 18:42:11 UTC

[gobblin] branch master updated: [GOBBLIN-1388] Fix bug when comparing partition type

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

suvasude 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 498911d  [GOBBLIN-1388] Fix bug when comparing partition type
498911d is described below

commit 498911de482f41dfd10eddc1a3b66c000f63c464
Author: Jack Moseley <jm...@linkedin.com>
AuthorDate: Wed Feb 17 10:42:05 2021 -0800

    [GOBBLIN-1388] Fix bug when comparing partition type
    
    Closes #3227 from jack-moseley/partition-type-bug
---
 .../gobblin/service/modules/dataset/FSDatasetPartitionConfig.java       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gobblin-service/src/main/java/org/apache/gobblin/service/modules/dataset/FSDatasetPartitionConfig.java b/gobblin-service/src/main/java/org/apache/gobblin/service/modules/dataset/FSDatasetPartitionConfig.java
index 737f844..c3eaee4 100644
--- a/gobblin-service/src/main/java/org/apache/gobblin/service/modules/dataset/FSDatasetPartitionConfig.java
+++ b/gobblin-service/src/main/java/org/apache/gobblin/service/modules/dataset/FSDatasetPartitionConfig.java
@@ -133,7 +133,7 @@ public class FSDatasetPartitionConfig {
       return false;
     }
     return ((DatasetDescriptorConfigKeys.DATASET_DESCRIPTOR_CONFIG_ANY.equalsIgnoreCase(getPartitionType())
-        || this.getPartitionType().equalsIgnoreCase(partitionType)))
+        || this.getPartitionType().equalsIgnoreCase(other.getPartitionType())))
         && ((DatasetDescriptorConfigKeys.DATASET_DESCRIPTOR_CONFIG_ANY.equalsIgnoreCase(getPartitionPattern())
         || this.getPartitionPattern().equalsIgnoreCase(other.getPartitionPattern())));
   }