ATLAS-3381: Atlas Patch Handler: Patch Handlers Not Committing Changes #2…

ATLAS-3381: Atlas Patch Handler: Patch Handlers Not Committing Changes #2 Improved check to detect classification presence Signed-off-by: Sarath Subramanian <sarath@apache.org>

ATLAS-3381: Atlas Patch Handler: Patch Handlers Not Committing Changes #2…
0d5c2229 · Ashutosh Mestry · Sarath Subramanian · 952d0696 · 0d5c2229
Commit 0d5c2229 authored Aug 20, 2019 by Ashutosh Mestry Committed by Sarath Subramanian Aug 21, 2019
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

EntityGraphMapper.java ...he/atlas/repository/store/graph/v2/EntityGraphMapper.java +3 -1

No files found.
--- a/repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphMapper.java
+++ b/repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphMapper.java
@@ -36,6 +36,7 @@ import org.apache.atlas.model.instance.EntityMutationResponse;
 import org.apache.atlas.model.instance.EntityMutations.EntityOperation;
 import org.apache.atlas.model.typedef.AtlasStructDef.AtlasAttributeDef;
 import org.apache.atlas.model.typedef.AtlasStructDef.AtlasAttributeDef.Cardinality;
+import org.apache.atlas.repository.Constants;
 import org.apache.atlas.repository.RepositoryException;
 import org.apache.atlas.repository.converters.AtlasInstanceConverter;
 import org.apache.atlas.repository.graph.FullTextMapperV2;
@@ -1697,7 +1698,8 @@ public class EntityGraphMapper {
    }
    public void updateClassificationTextAndNames(AtlasVertex vertex) throws AtlasBaseException {
-        if (StringUtils.isEmpty(vertex.getProperty(CLASSIFICATION_NAMES_KEY, String.class))) {
+        if(CollectionUtils.isEmpty(vertex.getPropertyValues(Constants.TRAIT_NAMES_PROPERTY_KEY, String.class)) &&
+                CollectionUtils.isEmpty(vertex.getPropertyValues(Constants.PROPAGATED_TRAIT_NAMES_PROPERTY_KEY, String.class))) {
            return;
        }