> On May 11, 2018, 10:34 a.m., David Radley wrote: > > omag-server/src/main/java/org/apache/atlas/omag/application/OMAGServerAdminResource.java > > Lines 510 (patched) > > <https://reviews.apache.org/r/67083/diff/1/?file=2019875#file2019875line510> > > > > OMAGInvalidParameterException is a checked excpetion so would need to > > be specified on the method signature. I think we need to remove all of the > > comments indicating that the this method can throw exceptions as it can now > > only return the void response, which contains the error details.
It is wrong to remove the comments about the exceptions that are passed in the response object. - Mandy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67083/#review202910 ----------------------------------------------------------- On May 16, 2018, 3:49 p.m., Bogdan Sava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67083/ > ----------------------------------------------------------- > > (Updated May 16, 2018, 3:49 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/4/ > > > 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 > >