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 60783: [ATLAS-1943] : Fix IT failure due to incorrect inverse reference check using relationship
Date Wed, 12 Jul 2017 21:58:13 GMT

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


Ship it!




Ship It!

- Madhan Neethiraj


On July 11, 2017, 8:14 p.m., Sarath Subramanian wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60783/
> -----------------------------------------------------------
> 
> (Updated July 11, 2017, 8:14 p.m.)
> 
> 
> Review request for atlas and Madhan Neethiraj.
> 
> 
> Bugs: ATLAS-1943
>     https://issues.apache.org/jira/browse/ATLAS-1943
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> Fix the following IT failures due to
> Failed tests: 
> EntityJerseyResourceIT.testCompleteUpdate:952 » AtlasService Metadata service ...
> EntityJerseyResourceIT.testPartialUpdate:877 » AtlasService Metadata service A...
> After creating inverse reference, call is made to create relationship for ownedRef attribute.
This call fails when an edge already exists. The edge exists as it was already created. Added
getOrCreate relationship method to get a relationship if it exists, if not create a new relationship.
> 
> 
> Diffs
> -----
> 
>   repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasRelationshipStore.java
341711ac 
>   repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasRelationshipStoreV1.java
8d9e4be8 
>   repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java
d5c1e862 
> 
> 
> Diff: https://reviews.apache.org/r/60783/diff/1/
> 
> 
> Testing
> -------
> 
> mvn clean package suceeded.
> 
> 
> Thanks,
> 
> Sarath Subramanian
> 
>


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