> On Sept. 10, 2015, 1:19 p.m., Rajat Khandelwal wrote: > > lens-api/src/main/java/org/apache/lens/api/query/save/ListResponse.java, > > line 30 > > <https://reviews.apache.org/r/37934/diff/6/?file=1066442#file1066442line30> > > > > This response can be generified to anything that is supposed to return > > a list. > > > > Can we probably make this `ListResponse<T>` and use > > `ListResponse<SavedQuery>` wherever required in this patch. > > Amruth Sampath wrote: > There is an issue when we try to use generics and marshalling. The > default MoxYJsonProvider seem to run into a bug whenever it encounters > generics (whenever accept header is JSON). > > Existing APIs like run which results a LensAPIResult<T> is also failing > when the accept header is JSON.
Unaware of the issue. Let's file a jira with steps to reproduce. > On Sept. 10, 2015, 1:19 p.m., Rajat Khandelwal wrote: > > lens-api/src/main/java/org/apache/lens/api/query/save/Parameter.java, line > > 42 > > <https://reviews.apache.org/r/37934/diff/6/?file=1066443#file1066443line42> > > > > final? > > Amruth Sampath wrote: > I need getters and setters for jaxb to do marshalling and un marshalling > (?). If I have to keep the object immutable, I need to have XML entities for > each model entity. Can I do these changes ? understood. It's okay if jaxb requires it. > On Sept. 10, 2015, 1:19 p.m., Rajat Khandelwal wrote: > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/SavedQueryHelper.java, > > lines 112-113 > > <https://reviews.apache.org/r/37934/diff/6/?file=1066451#file1066451line112> > > > > Validations should be on constructor level. > > Amruth Sampath wrote: > Again this is possible if I can have user facing entities for every model > entity I have and let JAXB do the marsh/unmarsh for user facing entities. I > can convert user facing entity to model entity internally. Should I proceed > doing these changes? I don't suppose there's a way to impose restrictions purely thorugh java annotations. If something can be found, that'd be great. - Rajat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37934/#review98361 ----------------------------------------------------------- On Sept. 10, 2015, 12:53 a.m., Amruth Sampath wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37934/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2015, 12:53 a.m.) > > > Review request for lens, Amareshwari Sriramadasu, Pranav Agarwal, Rajat > Khandelwal, and sharad agarwal. > > > Repository: lens > > > Description > ------- > > Refer to the JIRA description - https://issues.apache.org/jira/browse/LENS-742 > (Note : Sharing and CLI are not a part of this patch) > > > Diffs > ----- > > lens-api/src/main/java/org/apache/lens/api/error/LensCommonErrorCode.java > 754e6e1 > lens-api/src/main/java/org/apache/lens/api/query/save/ListResponse.java > PRE-CREATION > lens-api/src/main/java/org/apache/lens/api/query/save/Parameter.java > PRE-CREATION > > lens-api/src/main/java/org/apache/lens/api/query/save/ParameterCollectionType.java > PRE-CREATION > > lens-api/src/main/java/org/apache/lens/api/query/save/ParameterDataType.java > PRE-CREATION > > lens-api/src/main/java/org/apache/lens/api/query/save/ParameterParserResponse.java > PRE-CREATION > > lens-api/src/main/java/org/apache/lens/api/query/save/ResourceModifiedResponse.java > PRE-CREATION > lens-api/src/main/java/org/apache/lens/api/query/save/SavedQuery.java > PRE-CREATION > lens-api/src/main/resources/lens-errors.conf 5428041 > > lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java > fb11f93 > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/SavedQueryHelper.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/SavedQueryService.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/exception/MissingParameterException.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/exception/ParameterCollectionException.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/exception/ParameterValueException.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/exception/PrivilegeException.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/exception/SavedQueryNotFound.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/exception/ValueEncodeException.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/param/ParameterCollectionTypeEncoder.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/param/ParameterDataTypeEncoder.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/param/ParameterParser.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/param/ParameterResolver.java > PRE-CREATION > > lens-server-api/src/test/java/org/apache/lens/server/api/query/save/TestParameterParser.java > PRE-CREATION > > lens-server-api/src/test/java/org/apache/lens/server/api/query/save/TestParameterResolution.java > PRE-CREATION > lens-server/enunciate.xml 94b5199 > > lens-server/src/main/java/org/apache/lens/server/query/save/SavedQueryApp.java > PRE-CREATION > > lens-server/src/main/java/org/apache/lens/server/query/save/SavedQueryDao.java > PRE-CREATION > > lens-server/src/main/java/org/apache/lens/server/query/save/SavedQueryResource.java > PRE-CREATION > > lens-server/src/main/java/org/apache/lens/server/query/save/SavedQueryServiceImpl.java > PRE-CREATION > lens-server/src/main/java/org/apache/lens/server/util/UtilityMethods.java > 5d77eb7 > lens-server/src/main/resources/lensserver-default.xml 5d33eda > > lens-server/src/test/java/org/apache/lens/server/query/save/TestSavedQueryService.java > PRE-CREATION > lens-server/src/test/resources/lens-site.xml 4cf94d5 > src/site/apt/admin/config.apt b163a3a > src/site/apt/user/index.apt 6a86b1b > > Diff: https://reviews.apache.org/r/37934/diff/ > > > Testing > ------- > > Have added unit test cases for parsing, resolution of parameter and service > testing. > > > Thanks, > > Amruth Sampath > >
