atlas-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sarath Subramanian <sarath.ku...@gmail.com>
Subject Re: Review Request 56969: [ATLAS-1564] : EntityResource v1 should start routing its calls to v2 EntityREST
Date Fri, 24 Feb 2017 23:08:04 GMT

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

(Updated Feb. 24, 2017, 3:08 p.m.)


Review request for atlas, Apoorv Naik, Ashutosh Mestry, Madhan Neethiraj, Suma Shivaprasad,
and Vimal Sharma.


Changes
-------

addressed review comments.


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


Repository: atlas


Description
-------

With the v2 implementation of EntityREST ready, we must start routing all v1 entity CRUD calls
to start using v2 APIs in EntityREST.


Diffs (updated)
-----

  client/src/main/java/org/apache/atlas/AtlasClient.java 13896ce 
  intg/src/main/java/org/apache/atlas/AtlasErrorCode.java ca3023a 
  repository/src/main/java/org/apache/atlas/repository/converters/AtlasFormatConverter.java
9d0d7f4 
  repository/src/main/java/org/apache/atlas/repository/converters/AtlasInstanceConverter.java
e14fafb 
  repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java 3a037cc

  repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java
bbfc3e5 
  repository/src/main/java/org/apache/atlas/services/DefaultMetadataService.java 5127f74 
  server-api/src/main/java/org/apache/atlas/services/MetadataService.java e0fb66c 
  webapp/src/main/java/org/apache/atlas/web/resources/EntityResource.java f59cd9d 
  webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java 92ea93e 
  webapp/src/test/java/org/apache/atlas/web/resources/EntityResourceTest.java 3fe8e11 

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


Testing
-------

mvn clean package - all tests passed.


Thanks,

Sarath Subramanian


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message