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

Review request for atlas, Ashutosh Mestry, Jayendra Parab, Madhan Neethiraj, 
Pinal Shah, Radhika Kundam, Sarath Subramanian, Sheetal Shah, and Sidharth 
Mishra.


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


Repository: atlas


Description
-------

When Relationship typedef is created without "relationshipCategory", the 
default "relationshipCategory" provided as "ASSOCIATION".
For "relationshipCategory" "ASSOCIATION", it is required that both enddefs 
should have "isContainer" value as "false".

while creating the Relationship type, Whenever user does not provide the 
"relationshipCategory" value and put one of the "isContainer" value as "true".
Because of this, Relationship type gets created with "relationshipCategory" = 
"ASSOCIATION" and one of the Enddefs with "isContainer" = "true"

When atlas service is restarted, it throws and exception for such Relationship 
type. and due to this, entire TypeRegistry does not get populated.
So even after the Atlas service comes up, user cannot see any Typedefs.

At the time of create operation, this patch checks Relationship type which has 
no "relationshipCategory", it also checks for the "isContainer" values for both 
the Enddefs.
If both "isContainer" values are false, then only it sets the default 
"relationshipCategory" as "ASSOCIATION" otherwise it throws an exception.


Diffs
-----

  intg/src/main/java/org/apache/atlas/AtlasErrorCode.java 608342433 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasRelationshipDefStoreV2.java
 7bc4b28c3 


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


Testing
-------


Thanks,

Mandar Ambawane

Reply via email to