You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@atlas.apache.org by Suma Shivaprasad <su...@gmail.com> on 2017/02/02 09:20:54 UTC
Review Request 56206: ATLAS-1522 Entity V2 API changes to accept only
AtlasObjectId for child references
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/
-----------------------------------------------------------
Review request for atlas.
Bugs: ATLAS-1522
https://issues.apache.org/jira/browse/ATLAS-1522
Repository: atlas
Description
-------
Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
Diffs
-----
client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 2848a20
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 1590aee
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 31a619e
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
Diff: https://reviews.apache.org/r/56206/diff/
Testing
-------
Thanks,
Suma Shivaprasad
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/#review164051
-----------------------------------------------------------
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java (line 68)
<https://reviews.apache.org/r/56206/#comment235588>
It might be confusing to have "id" as a member here:
- information in "id" is already available in AtlasStruct.typeName and AtlasEntitiy.guid. Dealing with different value between these can be trouble and if possible should be avoided.
- AtlasObjectId supports uniqueAttributeValues to identify an object. This may not be useful/relevant here.
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java (line 133)
<https://reviews.apache.org/r/56206/#comment235597>
It might be useful to add isAssigned() and isUnAssigned() methods to AtlasObjectId(). The result can be computed once and cached.
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java (line 134)
<https://reviews.apache.org/r/56206/#comment235599>
Consider sending ((AtlasObjectId)entity) to addUnresolvedIdReference() - to avoid the method creating an instance of AtlasObjectId.
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java (line 78)
<https://reviews.apache.org/r/56206/#comment235601>
should vertex.guid be used in AtlasObjectId created here? entityId.getGuid() might have unassigned/null value - given this is about using unique attributes to find the entity.
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java (line 144)
<https://reviews.apache.org/r/56206/#comment235617>
Instead of iterating on all attributes, it might be quicker to iterate on entityId.getUniqueAttributes().
- Madhan Neethiraj
On Feb. 2, 2017, 9:37 a.m., Suma Shivaprasad wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56206/
> -----------------------------------------------------------
>
> (Updated Feb. 2, 2017, 9:37 a.m.)
>
>
> Review request for atlas.
>
>
> Bugs: ATLAS-1522
> https://issues.apache.org/jira/browse/ATLAS-1522
>
>
> Repository: atlas
>
>
> Description
> -------
>
> Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
>
>
> Diffs
> -----
>
> client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
> intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
> intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
> intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
> repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
> repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 2848a20
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 1590aee
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
> repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 31a619e
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
> webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
> webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
>
> Diff: https://reviews.apache.org/r/56206/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Suma Shivaprasad
>
>
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/#review164098
-----------------------------------------------------------
intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java (line 60)
<https://reviews.apache.org/r/56206/#comment235671>
@JsonIgore missing?
intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java (line 409)
<https://reviews.apache.org/r/56206/#comment235673>
Consider moving most of this method to AtlasObjectId.isValid(); and rewrite AtlasEntityType.validateAtlasObjectId(AtlasObjectId ) as:
class AtlasObjectId {
boolean isValid() {
if (StringUtils.isEmpty(typeName)) {
return false;
} else if (StringUtils.isNotEmpty(guid)) {
if (!isAssigned && !isUnassigned) {
return false;
}
} else if (MapUtils.isEmpty(uniqueAttributes)) {
return false;
}
return true;
}
}
class AtlasEntityType {
boolean isCompatibleObjectId(AtlasObjectId objId) {
return objId.isValid() && (StringUtils.equals(objId.getTypeName(), getTypeName()) || !isSuperTypeOf(objId.getTypeName()));
}
}
intg/src/main/java/org/apache/atlas/type/AtlasStructType.java (line 485)
<https://reviews.apache.org/r/56206/#comment235676>
Please add a comment here that "isContainedAttribute" can not be computed in the constructor - as structType is not fully populated at the time AtlasAttribute is constructed.
repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java (line 40)
<https://reviews.apache.org/r/56206/#comment235679>
final
repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java (line 45)
<https://reviews.apache.org/r/56206/#comment235680>
if the entities are modified after adding to the set, accessing them could be error prone. Please review and if possible, use a list instead.
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java (line 99)
<https://reviews.apache.org/r/56206/#comment235685>
Shouldn't this look for subType entities as well?
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapEntityStream.java (line 1)
<https://reviews.apache.org/r/56206/#comment235672>
missing license text
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java (line 129)
<https://reviews.apache.org/r/56206/#comment235686>
if vertex is not found by typeName, shouldn't we search for vertices that have typeName as a superType?
- Madhan Neethiraj
On Feb. 3, 2017, 7:26 a.m., Suma Shivaprasad wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56206/
> -----------------------------------------------------------
>
> (Updated Feb. 3, 2017, 7:26 a.m.)
>
>
> Review request for atlas.
>
>
> Bugs: ATLAS-1522
> https://issues.apache.org/jira/browse/ATLAS-1522
>
>
> Repository: atlas
>
>
> Description
> -------
>
> Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
>
>
> Diffs
> -----
>
> client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java 932a40d
> intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 738f22f
> intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
> intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
> intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93
> intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
> repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
> repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 0e1d9e6
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 4c79cef
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java e534d4f
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java f6e5055
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityStream.java PRE-CREATION
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapEntityStream.java PRE-CREATION
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
> repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 546cd0c
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java e3b4efa
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java da71c31
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 4f22437
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java 079f3be
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java f390e82
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectidConverter.java PRE-CREATION
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 382d1ef
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 4b3dd4c
> webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
> webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
> webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java d9bffe9
>
> Diff: https://reviews.apache.org/r/56206/diff/
>
>
> Testing
> -------
>
> ENtityJerseyResourceIt tests failing . This is pending.
>
>
> Thanks,
>
> Suma Shivaprasad
>
>
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/#review164209
-----------------------------------------------------------
Ship it!
Ship It!
- Madhan Neethiraj
On Feb. 4, 2017, 12:09 a.m., Suma Shivaprasad wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56206/
> -----------------------------------------------------------
>
> (Updated Feb. 4, 2017, 12:09 a.m.)
>
>
> Review request for atlas.
>
>
> Bugs: ATLAS-1522
> https://issues.apache.org/jira/browse/ATLAS-1522
>
>
> Repository: atlas
>
>
> Description
> -------
>
> Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
>
>
> Diffs
> -----
>
> client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
> intg/src/main/java/org/apache/atlas/AtlasErrorCode.java 0fb16c6
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java 932a40d
> intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 738f22f
> intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
> intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
> intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93
> intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
> repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
> repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 0e1d9e6
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 4c79cef
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java e534d4f
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java f6e5055
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityStream.java PRE-CREATION
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapEntityStream.java PRE-CREATION
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
> repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 546cd0c
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java e3b4efa
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java da71c31
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 4f22437
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java 079f3be
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java f390e82
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectIdConverter.java PRE-CREATION
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 382d1ef
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 4b3dd4c
> webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
> webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
> webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java d9bffe9
>
> Diff: https://reviews.apache.org/r/56206/diff/
>
>
> Testing
> -------
>
> ENtityJerseyResourceIt tests failing . This is pending.
>
>
> Thanks,
>
> Suma Shivaprasad
>
>
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Jeff Hagelberg <jn...@us.ibm.com>.
> On Feb. 4, 2017, 6:39 p.m., Suma Shivaprasad wrote:
> > Ship It!
How will this work for composite references? At least in V0 (I guess we can call it that), objects referenced via a composite reference need to be inlined.
- Jeff
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/#review164250
-----------------------------------------------------------
On Feb. 4, 2017, 12:09 a.m., Suma Shivaprasad wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56206/
> -----------------------------------------------------------
>
> (Updated Feb. 4, 2017, 12:09 a.m.)
>
>
> Review request for atlas.
>
>
> Bugs: ATLAS-1522
> https://issues.apache.org/jira/browse/ATLAS-1522
>
>
> Repository: atlas
>
>
> Description
> -------
>
> Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
>
>
> Diffs
> -----
>
> client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
> intg/src/main/java/org/apache/atlas/AtlasErrorCode.java 0fb16c6
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java 932a40d
> intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 738f22f
> intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
> intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
> intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93
> intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
> repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
> repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 0e1d9e6
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 4c79cef
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java e534d4f
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java f6e5055
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityStream.java PRE-CREATION
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapEntityStream.java PRE-CREATION
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
> repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 546cd0c
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java e3b4efa
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java da71c31
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 4f22437
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java 079f3be
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java f390e82
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectIdConverter.java PRE-CREATION
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 382d1ef
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 4b3dd4c
> webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
> webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
> webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java d9bffe9
>
> Diff: https://reviews.apache.org/r/56206/diff/
>
>
> Testing
> -------
>
> ENtityJerseyResourceIt tests failing . This is pending.
>
>
> Thanks,
>
> Suma Shivaprasad
>
>
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Suma Shivaprasad <su...@gmail.com>.
> On Feb. 4, 2017, 6:39 p.m., Suma Shivaprasad wrote:
> > Ship It!
>
> Jeff Hagelberg wrote:
> How will this work for composite references? At least in V0 (I guess we can call it that), objects referenced via a composite reference need to be inlined.
This is being modelled by AtlasEntityWithExtInfo which could have AtlasEntity objects which will have only AtlasObjectId references. Thse child references would be available in referredEntities and can be retrived using their guid/transient id.
- Suma
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/#review164250
-----------------------------------------------------------
On Feb. 4, 2017, 12:09 a.m., Suma Shivaprasad wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56206/
> -----------------------------------------------------------
>
> (Updated Feb. 4, 2017, 12:09 a.m.)
>
>
> Review request for atlas.
>
>
> Bugs: ATLAS-1522
> https://issues.apache.org/jira/browse/ATLAS-1522
>
>
> Repository: atlas
>
>
> Description
> -------
>
> Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
>
>
> Diffs
> -----
>
> client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
> intg/src/main/java/org/apache/atlas/AtlasErrorCode.java 0fb16c6
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java 932a40d
> intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 738f22f
> intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
> intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
> intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93
> intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
> repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
> repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 0e1d9e6
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 4c79cef
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java e534d4f
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java f6e5055
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityStream.java PRE-CREATION
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapEntityStream.java PRE-CREATION
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
> repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 546cd0c
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java e3b4efa
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java da71c31
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 4f22437
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java 079f3be
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java f390e82
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectIdConverter.java PRE-CREATION
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 382d1ef
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 4b3dd4c
> webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
> webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
> webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java d9bffe9
>
> Diff: https://reviews.apache.org/r/56206/diff/
>
>
> Testing
> -------
>
> ENtityJerseyResourceIt tests failing . This is pending.
>
>
> Thanks,
>
> Suma Shivaprasad
>
>
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Suma Shivaprasad <su...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/#review164250
-----------------------------------------------------------
Ship it!
Ship It!
- Suma Shivaprasad
On Feb. 4, 2017, 12:09 a.m., Suma Shivaprasad wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56206/
> -----------------------------------------------------------
>
> (Updated Feb. 4, 2017, 12:09 a.m.)
>
>
> Review request for atlas.
>
>
> Bugs: ATLAS-1522
> https://issues.apache.org/jira/browse/ATLAS-1522
>
>
> Repository: atlas
>
>
> Description
> -------
>
> Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
>
>
> Diffs
> -----
>
> client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
> intg/src/main/java/org/apache/atlas/AtlasErrorCode.java 0fb16c6
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
> intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java 932a40d
> intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 738f22f
> intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
> intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
> intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93
> intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
> repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
> repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 0e1d9e6
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 4c79cef
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java e534d4f
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java f6e5055
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityStream.java PRE-CREATION
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapEntityStream.java PRE-CREATION
> repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
> repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 546cd0c
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java e3b4efa
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java da71c31
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 4f22437
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java 079f3be
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java f390e82
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectIdConverter.java PRE-CREATION
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 382d1ef
> webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 4b3dd4c
> webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
> webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
> webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
> webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java d9bffe9
>
> Diff: https://reviews.apache.org/r/56206/diff/
>
>
> Testing
> -------
>
> ENtityJerseyResourceIt tests failing . This is pending.
>
>
> Thanks,
>
> Suma Shivaprasad
>
>
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Suma Shivaprasad <su...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/
-----------------------------------------------------------
(Updated Feb. 4, 2017, 12:09 a.m.)
Review request for atlas.
Changes
-------
Uploading Madhan's patch with some more fixes and optimizations
Bugs: ATLAS-1522
https://issues.apache.org/jira/browse/ATLAS-1522
Repository: atlas
Description
-------
Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
Diffs (updated)
-----
client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
intg/src/main/java/org/apache/atlas/AtlasErrorCode.java 0fb16c6
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java 932a40d
intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 738f22f
intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93
intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 0e1d9e6
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 4c79cef
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java e534d4f
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java f6e5055
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityStream.java PRE-CREATION
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapEntityStream.java PRE-CREATION
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 546cd0c
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java e3b4efa
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java da71c31
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 4f22437
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java 079f3be
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java f390e82
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectIdConverter.java PRE-CREATION
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 382d1ef
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 4b3dd4c
webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java d9bffe9
Diff: https://reviews.apache.org/r/56206/diff/
Testing
-------
ENtityJerseyResourceIt tests failing . This is pending.
Thanks,
Suma Shivaprasad
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Suma Shivaprasad <su...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/
-----------------------------------------------------------
(Updated Feb. 3, 2017, 7:02 p.m.)
Review request for atlas.
Changes
-------
Uploading Madhan's review comment fixes and other changes on top of my previous patch. Thanks Madhan
Bugs: ATLAS-1522
https://issues.apache.org/jira/browse/ATLAS-1522
Repository: atlas
Description
-------
Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
Diffs (updated)
-----
client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java 932a40d
intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 738f22f
intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93
intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 0e1d9e6
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 4c79cef
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java e534d4f
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java f6e5055
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityStream.java PRE-CREATION
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapEntityStream.java PRE-CREATION
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 546cd0c
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java e3b4efa
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java da71c31
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 4f22437
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java 079f3be
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java f390e82
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectIdConverter.java PRE-CREATION
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 382d1ef
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 4b3dd4c
webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java d9bffe9
Diff: https://reviews.apache.org/r/56206/diff/
Testing
-------
ENtityJerseyResourceIt tests failing . This is pending.
Thanks,
Suma Shivaprasad
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Suma Shivaprasad <su...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/
-----------------------------------------------------------
(Updated Feb. 3, 2017, 7:26 a.m.)
Review request for atlas.
Bugs: ATLAS-1522
https://issues.apache.org/jira/browse/ATLAS-1522
Repository: atlas
Description
-------
Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
Diffs (updated)
-----
client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java 932a40d
intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 738f22f
intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93
intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 0e1d9e6
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 4c79cef
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java e534d4f
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java f6e5055
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityStream.java PRE-CREATION
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapEntityStream.java PRE-CREATION
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 546cd0c
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java e3b4efa
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java da71c31
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 4f22437
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java 079f3be
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java f390e82
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectidConverter.java PRE-CREATION
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 382d1ef
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 4b3dd4c
webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java d9bffe9
Diff: https://reviews.apache.org/r/56206/diff/
Testing
-------
ENtityJerseyResourceIt tests failing . This is pending.
Thanks,
Suma Shivaprasad
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Suma Shivaprasad <su...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/
-----------------------------------------------------------
(Updated Feb. 3, 2017, 6:04 a.m.)
Review request for atlas.
Bugs: ATLAS-1522
https://issues.apache.org/jira/browse/ATLAS-1522
Repository: atlas
Description
-------
Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
Diffs
-----
client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java 932a40d
intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 738f22f
intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93
intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 2848a20
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 1590aee
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java e534d4f
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java f6e5055
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapSeekableInputStream.java PRE-CREATION
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 31a619e
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java e3b4efa
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java da71c31
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 4f22437
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java 079f3be
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java f390e82
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectidConverter.java PRE-CREATION
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 382d1ef
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 4b3dd4c
webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java d9bffe9
Diff: https://reviews.apache.org/r/56206/diff/
Testing (updated)
-------
ENtityJerseyResourceIt tests failing . This is pending.
Thanks,
Suma Shivaprasad
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Suma Shivaprasad <su...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/
-----------------------------------------------------------
(Updated Feb. 3, 2017, 6:03 a.m.)
Review request for atlas.
Bugs: ATLAS-1522
https://issues.apache.org/jira/browse/ATLAS-1522
Repository: atlas
Description
-------
Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
Diffs (updated)
-----
client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java 932a40d
intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java 738f22f
intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93
intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 2848a20
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 1590aee
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java e534d4f
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java f6e5055
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapSeekableInputStream.java PRE-CREATION
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 31a619e
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java e3b4efa
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java da71c31
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 4f22437
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java 079f3be
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java f390e82
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectidConverter.java PRE-CREATION
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 382d1ef
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 4b3dd4c
webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java d9bffe9
Diff: https://reviews.apache.org/r/56206/diff/
Testing
-------
Thanks,
Suma Shivaprasad
Re: Review Request 56206: ATLAS-1522 Entity V2 API changes to accept
only AtlasObjectId for child references
Posted by Suma Shivaprasad <su...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56206/
-----------------------------------------------------------
(Updated Feb. 2, 2017, 9:37 a.m.)
Review request for atlas.
Bugs: ATLAS-1522
https://issues.apache.org/jira/browse/ATLAS-1522
Repository: atlas
Description
-------
Currently child(composite) entities are allowed as attribute values in create or update APIs. Allowing this to have only AtlasObjectId would avoid cycles in the entity structure
Diffs (updated)
-----
client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7
intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4
intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java c9b6e97
intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c
intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java c42f95f
repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java 2d748da
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java 2848a20
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 1590aee
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java a989f76
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java d02b5a1
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java 76308a0
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java 31a619e
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java bbfcc59
webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 7368c72
webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767
webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java cfe9909
webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java ee36fdf
Diff: https://reviews.apache.org/r/56206/diff/
Testing
-------
Thanks,
Suma Shivaprasad