----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67083/#review203328 -----------------------------------------------------------
omrs/src/main/java/org/apache/atlas/omrs/admin/OMRSConfigurationFactory.java Line 55 (original), 55 (patched) <https://reviews.apache.org/r/67083/#comment285453> This should be plural to be consistent with access-services. there are a few otehr instances of this in the patch omrs/src/main/java/org/apache/atlas/omrs/admin/OMRSConfigurationFactory.java Line 291 (original), 291 (patched) <https://reviews.apache.org/r/67083/#comment285454> this is not what the code does. For this example wouldn't the code would produce urls starting with. ocalhost:8080/open-metadata/localServerName/open-metadata/repository-service/ omrs/src/main/java/org/apache/atlas/omrs/admin/OMRSConfigurationFactory.java Line 306 (original), 306 (patched) <https://reviews.apache.org/r/67083/#comment285455> should be repository-services to be consistant omrs/src/main/java/org/apache/atlas/omrs/rest/server/OMRSRepositoryRESTServices.java Line 202 (original), 201 (patched) <https://reviews.apache.org/r/67083/#comment285456> Are we missing the /users/ before the {userId} in the path? The same for the rest of this file. omrs/src/main/java/org/apache/atlas/omrs/rest/server/spring/OMRSRepositoryResource.java Lines 84 (patched) <https://reviews.apache.org/r/67083/#comment285457> plural for consistancy ? webapp/src/main/webapp/WEB-INF/openMetadataContext.xml Lines 3 (patched) <https://reviews.apache.org/r/67083/#comment285452> Bad end of line character on this and the next 6 lines - David Radley On May 17, 2018, 12:10 p.m., Bogdan Sava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67083/ > ----------------------------------------------------------- > > (Updated May 17, 2018, 12:10 p.m.) > > > Review request for atlas, Daniela Otelea, David Radley, Graham Wallis, Mandy > Chessell, and Ruxandra Rosu. > > > Repository: atlas > > > Description > ------- > > Create configuration for OMAG API in Atlas > > > Diffs > ----- > > om-fwk-ocf/pom.xml 8b1cefd > omag-api/pom.xml 2338286 > > omag-api/src/main/java/org/apache/atlas/omag/admin/OMAGServerAdministration.java > dd19018 > omag-api/src/main/java/org/apache/atlas/omag/admin/OMAGServerCategory.java > PRE-CREATION > > omag-api/src/main/java/org/apache/atlas/omag/admin/server/OMAGServerAdminRESTServices.java > PRE-CREATION > > omag-api/src/main/java/org/apache/atlas/omag/admin/server/OMAGServerAdminResource.java > ebc9f2a > > omag-api/src/main/java/org/apache/atlas/omag/admin/server/spring/OMAGServerAdminResource.java > PRE-CREATION > > omag-api/src/main/java/org/apache/atlas/omag/configuration/registration/AccessServiceDescription.java > 1a15215 > > omag-api/src/main/java/org/apache/atlas/omag/configuration/registration/AccessServiceRegistration.java > c2950c4 > omag-server/pom.xml 527c5c3 > > omag-server/src/main/java/org/apache/atlas/omag/application/OMAGApplication.java > 88397de > omag-server/src/main/resources/application.properties PRE-CREATION > omas-assetconsumer/pom.xml 29935e7 > > omas-assetconsumer/src/main/java/org/apache/atlas/omas/assetconsumer/client/AssetConsumer.java > 80a98f5 > > omas-assetconsumer/src/main/java/org/apache/atlas/omas/assetconsumer/publisher/AssetConsumerPublisher.java > > > omas-assetconsumer/src/main/java/org/apache/atlas/omas/assetconsumer/server/AssetConsumerRESTServices.java > 967be5a > > omas-assetconsumer/src/main/java/org/apache/atlas/omas/assetconsumer/server/spring/AssetConsumerOMASResource.java > PRE-CREATION > omas-connectedasset/README.md 2f431df > omas-connectedasset/pom.xml 219c972 > > omas-connectedasset/src/main/java/org/apache/atlas/omas/connectedasset/server/ConnectedAssetRESTServices.java > 8ae6c37 > > omas-connectedasset/src/main/java/org/apache/atlas/omas/connectedasset/server/spring/ConnectedAssetOMASResource.java > PRE-CREATION > omrs/pom.xml 50b9898 > > omrs/src/main/java/org/apache/atlas/omrs/admin/OMRSConfigurationFactory.java > 7ae87c6 > > omrs/src/main/java/org/apache/atlas/omrs/rest/server/OMRSRepositoryRESTServices.java > 5948482 > > omrs/src/main/java/org/apache/atlas/omrs/rest/server/spring/OMRSRepositoryResource.java > PRE-CREATION > > omrs/src/test/java/org/apache/atlas/omrs/archivemanager/opentypes/TestOpenMetadataTypesArchive.java > 3f85317 > > omrs/src/test/java/org/apache/atlas/omrs/archivemanager/store/TestOMRSArchiveBuilder.java > ccf0de6 > webapp/pom.xml 03b8408 > webapp/src/main/webapp/WEB-INF/applicationContext.xml 2ecd8d1 > webapp/src/main/webapp/WEB-INF/openMetadataContext.xml PRE-CREATION > webapp/src/main/webapp/WEB-INF/web.xml 23dc063 > > > Diff: https://reviews.apache.org/r/67083/diff/5/ > > > Testing > ------- > > Create configuration for OMAG API in Atlas using Spring MVC Dispatcher > servlet. > > > Change base URL for the API to "/open-metadata/access-services" > > > Clean maven unused dependenies > > The Jira issue is https://issues.apache.org/jira/browse/ATLAS-2665 > > > File Attachments > ---------------- > > Added apache licence to a properties file > > https://reviews.apache.org/media/uploaded/files/2018/05/11/77fbc4f6-b16a-42f9-983c-261f70f3ca9f__0002-ATLAS-2665-Added-apache-licence-to-properties-file.patch > 0003-ATLAS-2665-comments-fixed.patch > > https://reviews.apache.org/media/uploaded/files/2018/05/11/004f018b-1ee5-4da1-91d4-b50f4913298d__0003-ATLAS-2665-comments-fixed.patch > ATLAS-2665-Add-OMAG-API-in-Atlas-v2.patch > > https://reviews.apache.org/media/uploaded/files/2018/05/14/160f8499-661d-4fa9-8ce1-afe4bd88b723__ATLAS-2665-Add-OMAG-API-in-Atlas-v2.patch > > > Thanks, > > Bogdan Sava > >