You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@atlas.apache.org by Deep Singh <de...@gmail.com> on 2020/09/30 22:50:07 UTC

Re: Review Request 72920: ATLAS-3965:Relationships with null values are missing in entity relationshipAttributes

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72920/
-----------------------------------------------------------

(Updated Sept. 30, 2020, 10:50 p.m.)


Review request for atlas, Ashutosh Mestry, madhan, and Sarath Subramanian.


Bugs: ATLAS-3965
    https://issues.apache.org/jira/browse/ATLAS-3965


Repository: atlas


Description
-------

In the get call, while reading the entity, we map entity's relationshipAttributes by relationship attributes defined in entityTypeDef.
At this stage we ignore all null valued attributes, therefore all undefined attributes with Single cardinality gets ignored. However, attributes with Set or List cardinality are not ignored as they are not null by default but empty Arrays.

While mapping relationshipAttributes from entityTypeDef to response-entity, we should either ignore empty arrays as well or not ignore anything. In this Patch I have followed the second aproach which is not to ignore any thing.


Diffs
-----

  repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphRetriever.java 00146b106 


Diff: https://reviews.apache.org/r/72920/diff/1/


Testing
-------

Testing is done manually by importing Kafka topic.
PreCommit Build Test has also passed. 
https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/44//


Thanks,

Deep Singh