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

(Updated Feb. 7, 2017, 10:55 p.m.)


Review request for atlas.


Changes
-------

updated patch after merge with latest master and addressing the review comments


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


Repository: atlas


Description
-------

constraints foreignKey/mappedFromRef introduced in v2 doesn't handle all use 
cases of legacy flags isComposite/reverseAttributeName. Hence replace these 
constrainsts with new constraints "owner" and "inverserRef(attribute=attr)"


Diffs (updated)
-----

  addons/models/0030-hive_model.json 5ae16cf 
  addons/models/0060-hbase_model.json 5febbbc 
  addons/models/0080-storm_model.json 344227d 
  intg/src/main/java/org/apache/atlas/AtlasErrorCode.java 49289d8 
  intg/src/main/java/org/apache/atlas/model/typedef/AtlasStructDef.java 2c00f54 
  intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java 761eef5 
  intg/src/main/java/org/apache/atlas/type/AtlasStructType.java f444e93 
  intg/src/test/java/org/apache/atlas/TestUtilsV2.java b4aa8aa 
  intg/src/test/java/org/apache/atlas/type/TestAtlasEntityType.java 032dce0 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasStructDefStoreV1.java
 69cf60a 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java
 f507ae0 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/StructVertexMapper.java
 2e9e5e4 
  webapp/src/main/java/org/apache/atlas/examples/QuickStartV2.java d8202a5 
  webapp/src/main/java/org/apache/atlas/util/RestUtils.java 9d4b04e 
  
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java
 4158f01 
  webapp/src/test/java/org/apache/atlas/web/resources/BaseResourceIT.java 
6187cfb 

Diff: https://reviews.apache.org/r/56380/diff/


Testing
-------

updated unit tests and verified that existing unit tests pass


Thanks,

Madhan Neethiraj

Reply via email to