atlas-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Madhan Neethiraj <mad...@apache.org>
Subject Re: Review Request 53316: Initial draft of changes for ATLAS Entities REST to talk to v1 backend
Date Fri, 11 Nov 2016 02:02:38 GMT

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




webapp/src/main/java/org/apache/atlas/web/adapters/v1/ReferenceableToAtlasEntityConverter.java
(line 65)
<https://reviews.apache.org/r/53316/#comment225756>

    If 'type' is of type AtlasEntityType, consider the following:
    
    AtlasEntityDef entityDef = ((AtlasEntityType)type).getEntityDef();



webapp/src/main/java/org/apache/atlas/web/adapters/v1/StructToAtlasStructConverter.java (line
64)
<https://reviews.apache.org/r/53316/#comment225755>

    If 'type' is of type AtlasStructType, consider the following:
    
    AtlasStructDef structDef = ((AtlasStructType)type).getStructDef();



webapp/src/main/java/org/apache/atlas/web/adapters/v2/AtlasEntityToReferenceableConverter.java
(line 87)
<https://reviews.apache.org/r/53316/#comment225757>

    isn't 'type' parameter same as the 'entityType' here? Please review here and line #78.



webapp/src/main/java/org/apache/atlas/web/adapters/v2/AtlasStructToStructConverter.java (line
80)
<https://reviews.apache.org/r/53316/#comment225759>

    Consider replacing with:
    
    AtlasStructDef structDef = ((AtlasStructType)type).getStructDef();


- Madhan Neethiraj


On Nov. 11, 2016, 12:12 a.m., Suma Shivaprasad wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/53316/
> -----------------------------------------------------------
> 
> (Updated Nov. 11, 2016, 12:12 a.m.)
> 
> 
> Review request for atlas.
> 
> 
> Bugs: ATLAS-1257
>     https://issues.apache.org/jira/browse/ATLAS-1257
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> EntitiesREST will talk to the ATLAS v1 backend by converting from 
> a. AtlasEntity to Referenceable 
> b. AtlasStruct to Struct
> c. Map of complex types
> d. Array of complex types
> and call the V1 backend directly as a workaround before v2 implementation is completely
ready.
> 
> 
> Diffs
> -----
> 
>   distro/src/bin/atlas_stop.py a2b1c31 
>   intg/src/main/java/org/apache/atlas/AtlasErrorCode.java fe38fba 
>   intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 6727f27 
>   intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 5921fea 
>   intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java a6c75ed

>   intg/src/main/java/org/apache/atlas/type/AtlasTypeUtil.java a53b11d 
>   intg/src/test/java/org/apache/atlas/TestUtilsV2.java bae9d3b 
>   repository/src/main/java/org/apache/atlas/services/DefaultMetadataService.java 69e8d12

>   repository/src/test/java/org/apache/atlas/service/DefaultMetadataServiceTest.java 37c2374

>   server-api/src/main/java/org/apache/atlas/services/MetadataService.java 2351891 
>   webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java PRE-CREATION

>   webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java PRE-CREATION

>   webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatAdapter.java PRE-CREATION

>   webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverters.java PRE-CREATION

>   webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConvertersModule.java
PRE-CREATION 
>   webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java PRE-CREATION

>   webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java PRE-CREATION

>   webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java
PRE-CREATION 
>   webapp/src/main/java/org/apache/atlas/web/adapters/v1/ReferenceableToAtlasEntityConverter.java
PRE-CREATION 
>   webapp/src/main/java/org/apache/atlas/web/adapters/v1/StructToAtlasStructConverter.java
PRE-CREATION 
>   webapp/src/main/java/org/apache/atlas/web/adapters/v2/AtlasEntityToReferenceableConverter.java
PRE-CREATION 
>   webapp/src/main/java/org/apache/atlas/web/adapters/v2/AtlasStructToStructConverter.java
PRE-CREATION 
>   webapp/src/main/java/org/apache/atlas/web/listeners/GuiceServletConfig.java 0c2e0ec

>   webapp/src/main/java/org/apache/atlas/web/resources/EntityResource.java bbf01a6 
>   webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 543cbe2 
>   webapp/src/main/java/org/apache/atlas/web/rest/EntityRest.java df5138e 
>   webapp/src/test/java/org/apache/atlas/web/adapters/TestAtlasEntitiesREST.java PRE-CREATION

>   webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java PRE-CREATION

> 
> Diff: https://reviews.apache.org/r/53316/diff/
> 
> 
> Testing
> -------
> 
> Pending - UTS in progress
> REST layer - tested create and update through the new APIs
> 
> 
> Thanks,
> 
> Suma Shivaprasad
> 
>


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