----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52995/#review153487 -----------------------------------------------------------
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasTypeDefGraphStore.java (line 120) <https://reviews.apache.org/r/52995/#comment222849> Not finding any enum does not seem to be an error condition. Consider returning an empty list instead (i.e. not null). Please review other such getAll*() and search*() methods as well. webapp/src/main/java/org/apache/atlas/web/errors/LoggingExceptionMapper.java (line 42) <https://reviews.apache.org/r/52995/#comment222852> Instead of replacing generic type E with AtlasBaseException, it might be better to handle AtlasBaseException type of exception inside toResponse(E excp) - similar to how WebApplicationException is handled. Woud that make this mapper to handle more type of exceptions? Please review. webapp/src/main/java/org/apache/atlas/web/errors/LoggingExceptionMapper.java (line 56) <https://reviews.apache.org/r/52995/#comment222850> Consider replacing this with AtlasType.toJson(errorJsonMap) - Madhan Neethiraj On Oct. 20, 2016, 5:29 p.m., Apoorv Naik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52995/ > ----------------------------------------------------------- > > (Updated Oct. 20, 2016, 5:29 p.m.) > > > Review request for atlas, Madhan Neethiraj, Sarath Subramanian, Shwetha GS, > and Suma Shivaprasad. > > > Bugs: ATLAS-1200 > https://issues.apache.org/jira/browse/ATLAS-1200 > > > Repository: atlas > > > Description > ------- > > Enhanced AtlasBaseException class to accept AtlasErrorCode enum and > parameters for the formatted error message. > AtlasErrorCode enum captures all AtlasErrors identified by a unique code such > as AtlasE4001 etc. > > Each error code has a error message/description which is mapped under > errorMessage.properties (for I18N) and can be parameterized to provide more > context about the failure. > > > Diffs > ----- > > common/src/main/java/org/apache/atlas/AtlasConstants.java > 8521f630e202adc3024d7fd7a94a76c5a52771fa > intg/src/main/java/org/apache/atlas/AtlasErrorCode.java PRE-CREATION > intg/src/main/java/org/apache/atlas/exception/AtlasBaseException.java > 3538f41355b23b60e79be4179cc7a05100bd7817 > intg/src/main/java/org/apache/atlas/model/typedef/AtlasTypesDef.java > 5d3f4a38a7a34a973dc7bd43c3913c7b7925265b > > repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasTypeDefGraphStore.java > 12047bfd80c6fb37b61dfd401b35f9e3dfb41043 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasClassificationDefStoreV1.java > 7b6e08a8b86889b64c57c82d53c55d0a9370e3b1 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityDefStoreV1.java > 308a5294607ffb7d8c5938b9cde9f828da5043ee > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEnumDefStoreV1.java > 5c55d421a43a3579595b76a84360b4cd5dfcc7c9 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasStructDefStoreV1.java > eeb08d0a4bdb4592e4837df410eb3097bd55d26a > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasTypeDefGraphStoreV1.java > 663470790dda87ea982ab63f22b94d3d576fd4ee > > webapp/src/main/java/org/apache/atlas/web/errors/LoggingExceptionMapper.java > 9830269d35aed961f0eecdfa6a57edf93e982de5 > webapp/src/main/java/org/apache/atlas/web/rest/TypesREST.java > 065836143b3f2604ea6e3754069afea77e87f9c1 > webapp/src/main/webapp/WEB-INF/web.xml > c5518142492dab0e343caf4cedbe8eb1a645364a > > Diff: https://reviews.apache.org/r/52995/diff/ > > > Testing > ------- > > Tested will REST API calls. > Unit Tests coverage will be added as a part of the UT review request for > TypesREST stores. > > > Thanks, > > Apoorv Naik > >