> On Dec. 6, 2016, 2:22 a.m., Suma Shivaprasad wrote: > > webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java, line 197 > > <https://reviews.apache.org/r/54058/diff/4/?file=1573384#file1573384line197> > > > > add validations to have typename or supertype in the request to be > > provided mandatorily. Else it will end up running a query which may not be > > indexed.
I've added a todo section for this in the store class. - Apoorv ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54058/#review158102 ----------------------------------------------------------- On Nov. 30, 2016, 11:22 p.m., Apoorv Naik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54058/ > ----------------------------------------------------------- > > (Updated Nov. 30, 2016, 11:22 p.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 > ----- > > addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java > 23a13ea > client/src/main/java/org/apache/atlas/AtlasBaseClient.java 04a418a > client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java fae4dd8 > client/src/main/java/org/apache/atlas/AtlasServiceException.java 4719e7c > client/src/main/java/org/apache/atlas/AtlasTypedefClientV2.java cf86e7a > > intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java > 72f4118 > > repository/src/main/java/org/apache/atlas/repository/graph/GraphToTypedInstanceMapper.java > 84608d9 > webapp/src/main/java/org/apache/atlas/util/RestUtils.java 7597dcb > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java > 5756adb > webapp/src/main/java/org/apache/atlas/web/resources/BaseService.java > dfd29b1 > webapp/src/main/java/org/apache/atlas/web/resources/EntityResource.java > 8a663c2 > webapp/src/main/java/org/apache/atlas/web/resources/EntityService.java > ac4f9f1 > webapp/src/main/java/org/apache/atlas/web/resources/TaxonomyService.java > cc98207 > webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 768ef12 > webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java 6bbc69c > webapp/src/main/java/org/apache/atlas/web/rest/TypesREST.java 867307f > webapp/src/main/java/org/apache/atlas/web/rest/module/RestModule.java > 62e1e57 > webapp/src/main/resources/spring-security.xml 4ba3025 > webapp/src/main/webapp/WEB-INF/applicationContext.xml 6129605 > webapp/src/test/java/org/apache/atlas/examples/QuickStartIT.java c2f89bd > > webapp/src/test/java/org/apache/atlas/notification/EntityNotificationIT.java > 081d5ca > > webapp/src/test/java/org/apache/atlas/notification/NotificationHookConsumerIT.java > de0a459 > webapp/src/test/java/org/apache/atlas/web/resources/BaseResourceIT.java > 1777f75 > > webapp/src/test/java/org/apache/atlas/web/resources/DataSetLineageJerseyResourceIT.java > d9f14d0 > > webapp/src/test/java/org/apache/atlas/web/resources/EntityJerseyResourceIT.java > 38c823a > > webapp/src/test/java/org/apache/atlas/web/resources/EntityLineageJerseyResourceIT.java > cf25814 > > webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java > PRE-CREATION > > webapp/src/test/java/org/apache/atlas/web/resources/MetadataDiscoveryJerseyResourceIT.java > 13ef503 > > webapp/src/test/java/org/apache/atlas/web/resources/TypedefsJerseyResourceIT.java > 2fa1602 > > webapp/src/test/java/org/apache/atlas/web/resources/TypesJerseyResourceIT.java > fd15544 > > Diff: https://reviews.apache.org/r/54058/diff/ > > > Testing > ------- > > UTs and ITs are running fine. mvn clean install -Pdist was successful > > > Thanks, > > Apoorv Naik > >
