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

(Updated July 19, 2017, 12:27 p.m.)


Review request for atlas and Madhan Neethiraj.


Changes
-------

addressed review comments


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


Repository: atlas


Description
-------

* Improve relationship model to support create/update operations and the 
following cardinalities (previously supported using inverseReference):
1 to 1
1 to many
many to 1
many to many


* Change legacyLabel flag in AtlasRelationshipEndDef to boolean flag.
* Add unit tests for the above cases.


Diffs (updated)
-----

  addons/models/0010-base_model.json 303f3796 
  addons/models/0030-hive_model.json a795f0f3 
  addons/models/0050-falcon_model.json 7755fa86 
  addons/models/0060-hbase_model.json 1d264df4 
  addons/models/0080-storm_model.json 25360ff0 
  intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 68da6af1 
  intg/src/main/java/org/apache/atlas/model/instance/AtlasRelationship.java 
2de9bdf0 
  intg/src/main/java/org/apache/atlas/model/typedef/AtlasRelationshipDef.java 
fc820d49 
  
intg/src/main/java/org/apache/atlas/model/typedef/AtlasRelationshipEndDef.java 
f80ea895 
  intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java e94dd190 
  intg/src/main/java/org/apache/atlas/type/AtlasRelationshipType.java 841b66f7 
  intg/src/main/java/org/apache/atlas/type/AtlasStructType.java f97d7674 
  intg/src/test/java/org/apache/atlas/TestRelationshipUtilsV2.java PRE-CREATION 
  intg/src/test/java/org/apache/atlas/TestUtilsV2.java 9774583d 
  repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java 
6f6d74bc 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java
 12e8bb1f 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasGraphUtilsV1.java
 cd9a47ad 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasRelationshipStoreV1.java
 3ff6fbef 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java
 d4fdc257 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java
 157f8cd2 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphRetriever.java
 f4257be7 
  
repository/src/test/java/org/apache/atlas/repository/impexp/ImportServiceTest.java
 404225c2 
  
repository/src/test/java/org/apache/atlas/repository/impexp/ZipFileResourceTestUtils.java
 d9017319 
  
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasRelationshipStoreHardDeleteV1Test.java
 PRE-CREATION 
  
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasRelationshipStoreSoftDeleteV1Test.java
 PRE-CREATION 
  
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasRelationshipStoreV1Test.java
 67702231 


Diff: https://reviews.apache.org/r/60938/diff/4/

Changes: https://reviews.apache.org/r/60938/diff/3-4/


Testing
-------

added unit test - AtlasRelationshipStoreV1Test

mvn clean package - succeeded with no errors.


Thanks,

Sarath Subramanian

Reply via email to