commit fe86177a1d55b18b9b1a205fcd1cae83c6b85606
author Madhan Neethiraj <madhan@apache.org> Sun Mar 05 13:05:17 2017 -0800
committer Madhan Neethiraj <madhan@apache.org> Mon Mar 06 00:38:41 2017 -0800
tree bb0f378a893cc4d453fa491c238ad55e9fb00c9f
parent 42a441aa63d2015e75ddc783167aa1b817c5e829

ATLAS-1635: fixed v1 lineage API return to return correct value in qualifiedName attribute

repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphRetriever.java

diff --git a/repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphRetriever.java b/repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphRetriever.java
index 815c3d2..075b9b6 100644
--- a/repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphRetriever.java
+++ b/repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphRetriever.java
@@ -197,14 +197,18 @@
         AtlasEntityType entityType = typeRegistry.getEntityTypeByName(typeName);
 
         if (entityType != null) {
+            for (AtlasAttribute uniqueAttribute : entityType.getUniqAttributes().values()) {
+                Object attrValue = getVertexAttribute(entityVertex, uniqueAttribute);
+
+                if (attrValue != null) {
+                    ret.setAttribute(uniqueAttribute.getName(), attrValue);
+                }
+            }
+
             Object name        = getVertexAttribute(entityVertex, entityType.getAttribute(AtlasClient.NAME));
             Object description = getVertexAttribute(entityVertex, entityType.getAttribute(AtlasClient.DESCRIPTION));
             Object owner       = getVertexAttribute(entityVertex, entityType.getAttribute(AtlasClient.OWNER));
-            Object displayText = name;
-
-            if (displayText == null) {
-                displayText = getVertexAttribute(entityVertex, entityType.getAttribute(AtlasClient.QUALIFIED_NAME));
-            }
+            Object displayText = name != null ? name : ret.getAttribute(AtlasClient.QUALIFIED_NAME);
 
             ret.setAttribute(AtlasClient.NAME, name);
             ret.setAttribute(AtlasClient.DESCRIPTION, description);

webapp/src/main/java/org/apache/atlas/web/util/LineageUtils.java

diff --git a/webapp/src/main/java/org/apache/atlas/web/util/LineageUtils.java b/webapp/src/main/java/org/apache/atlas/web/util/LineageUtils.java
index 58ddf8b..ad2a697 100644
--- a/webapp/src/main/java/org/apache/atlas/web/util/LineageUtils.java
+++ b/webapp/src/main/java/org/apache/atlas/web/util/LineageUtils.java
@@ -72,8 +72,13 @@
                     vertexIdMap.put(idType.stateAttrName(), (entityHeader.getStatus() == AtlasEntity.Status.ACTIVE) ? "ACTIVE" : "DELETED");
                     vertexIdMap.put(idType.typeNameAttrName(), entityHeader.getTypeName());
 
+                    Object qualifiedName = entityHeader.getAttribute(AtlasClient.REFERENCEABLE_ATTRIBUTE_NAME);
+                    if (qualifiedName == null) {
+                        qualifiedName = entityHeader.getDisplayText();
+                    }
+
                     Map<String, Object> values = new HashMap<>();
-                    values.put(AtlasClient.REFERENCEABLE_ATTRIBUTE_NAME, entityHeader.getDisplayText());
+                    values.put(AtlasClient.REFERENCEABLE_ATTRIBUTE_NAME, qualifiedName);
                     values.put(VERTEX_ID_ATTR_NAME, constructResultStruct(vertexIdMap, true));
                     values.put(AtlasClient.NAME, entityHeader.getDisplayText());
                     verticesMap.put(guid, constructResultStruct(values, false));