Mailing-List: contact dev-help@atlas.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@atlas.incubator.apache.org
Content-Type: multipart/alternative;
 boundary="===============5703235881229612106=="
MIME-Version: 1.0
Subject: Re: Review Request 54058: ATLAS-1311: Integration tests for the V2
 Entity APIs
From: David Radley <david_radley@uk.ibm.com>
To: Sarath Subramanian <sarath.kum4r@gmail.com>,
 Madhan Neethiraj <madhan@apache.org>,
 Shwetha GS <sshivalingamurthy@hortonworks.com>
Cc: David Radley <david_radley@uk.ibm.com>, Apoorv Naik <naik.apoorv@gmail.com>,
 atlas <dev@atlas.incubator.apache.org>
Date: Thu, 22 Dec 2016 17:48:43 -0000
Message-ID: <20161222174843.1708.43128@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: David Radley <noreply@reviews.apache.org>
References: <20161221022547.1708.73221@reviews.apache.org>
In-Reply-To: <20161221022547.1708.73221@reviews.apache.org>
Reply-To: David Radley <david_radley@uk.ibm.com>
archived-at: Thu, 22 Dec 2016 17:48:53 -0000

--===============5703235881229612106==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


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


I am looking for tests that drive the Entity V2 API with updates to LIST and SET attributes for foreign keys and map from refs. It would be good to show how to update a LIST - for eaxmple can you add a member into the middle of the list. Is this be in scope for this Jira?

- David Radley


On Dec. 21, 2016, 2:25 a.m., Apoorv Naik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54058/
> -----------------------------------------------------------
> 
> (Updated Dec. 21, 2016, 2:25 a.m.)
> 
> 
> Review request for atlas, Madhan Neethiraj, Sarath Subramanian, Shwetha GS, and Suma Shivaprasad.
> 
> 
> Bugs: ATLAS-1311
>     https://issues.apache.org/jira/browse/ATLAS-1311
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> ATLAS-1311: Integration tests for the V2 Entity APIs
> 
> 
> Diffs
> -----
> 
>   .gitignore d2fbdc01 
>   addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java e5bd65a8 
>   client/src/main/java/org/apache/atlas/AtlasBaseClient.java a95bf32d 
>   client/src/main/java/org/apache/atlas/AtlasClient.java 47231a87 
>   client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 16556c8c 
>   client/src/main/java/org/apache/atlas/AtlasServiceException.java 4719e7cf 
>   client/src/main/java/org/apache/atlas/AtlasTypedefClientV2.java a1935247 
>   intg/src/main/java/org/apache/atlas/model/SearchFilter.java 4d8b258e 
>   intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java 35f2f144 
>   repository/src/main/java/org/apache/atlas/repository/graph/GraphToTypedInstanceMapper.java 1a3c31c9 
>   repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java 6b2b2166 
>   webapp/src/main/java/org/apache/atlas/util/RestUtils.java 7597dcbe 
>   webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java 5756adb5 
>   webapp/src/main/java/org/apache/atlas/web/resources/BaseService.java dfd29b14 
>   webapp/src/main/java/org/apache/atlas/web/resources/EntityResource.java 8a663c28 
>   webapp/src/main/java/org/apache/atlas/web/resources/EntityService.java ac4f9f1a 
>   webapp/src/main/java/org/apache/atlas/web/resources/TaxonomyService.java cc98207c 
>   webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 768ef12c 
>   webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java 2529f0dd 
>   webapp/src/main/resources/spring-security.xml 4ba30255 
>   webapp/src/main/webapp/WEB-INF/applicationContext.xml 6129605d 
>   webapp/src/test/java/org/apache/atlas/examples/QuickStartIT.java 01e4d483 
>   webapp/src/test/java/org/apache/atlas/notification/EntityNotificationIT.java ec62112f 
>   webapp/src/test/java/org/apache/atlas/notification/NotificationHookConsumerIT.java de0a459b 
>   webapp/src/test/java/org/apache/atlas/notification/NotificationHookConsumerKafkaTest.java e37839a2 
>   webapp/src/test/java/org/apache/atlas/web/resources/AdminJerseyResourceIT.java 2a4babae 
>   webapp/src/test/java/org/apache/atlas/web/resources/BaseResourceIT.java 325c25b4 
>   webapp/src/test/java/org/apache/atlas/web/resources/DataSetLineageJerseyResourceIT.java 7e4db13f 
>   webapp/src/test/java/org/apache/atlas/web/resources/EntityJerseyResourceIT.java 22bcc027 
>   webapp/src/test/java/org/apache/atlas/web/resources/EntityLineageJerseyResourceIT.java c681a0f9 
>   webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java PRE-CREATION 
>   webapp/src/test/java/org/apache/atlas/web/resources/MetadataDiscoveryJerseyResourceIT.java ed37c3a1 
>   webapp/src/test/java/org/apache/atlas/web/resources/TypesJerseyResourceIT.java 87bf9a89 
> 
> Diff: https://reviews.apache.org/r/54058/diff/
> 
> 
> Testing
> -------
> 
> UTs and ITs are running fine. mvn clean install -Pdist was successful
> 
> 
> Thanks,
> 
> Apoorv Naik
> 
>


--===============5703235881229612106==--