> On Jan. 15, 2015, 9:12 a.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryContext.java, > > line 118 > > <https://reviews.apache.org/r/29919/diff/1/?file=822465#file822465line118> > > > > Can you add comment for why Setter for groupbyAST, also?
setGroupbyAST is used in some other file elsewhere. - Rajat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29919/#review68225 ----------------------------------------------------------- On Jan. 19, 2015, 7:05 a.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29919/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2015, 7:05 a.m.) > > > Review request for lens. > > > Bugs: LENS-149 > https://issues.apache.org/jira/browse/LENS-149 > > > Repository: lens > > > Description > ------- > > 1. In the json output, showing error -> list of facts instead of list of fact > -> error. Many facts have same error, makes the error a bit compact. > 2. converting getters/setters to annotations. > > > Diffs > ----- > > lens-cube/src/main/java/org/apache/lens/cube/parse/AggregateResolver.java > 33a4a7c7357206d48dbc8c91360eb57851f9572d > > lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTablePruneCause.java > b346da6f2a0d18eb11a04a4c4e88d85ba4c2bff9 > > lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTableResolver.java > 4d472acfe2d0d0ef5d505bffc87c1261af76decc > lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryContext.java > 4a03701c73125a14f72378ab84234ad6563e6db8 > > lens-cube/src/main/java/org/apache/lens/cube/parse/DenormalizationResolver.java > 5643b361bc0fdd4571ad7ac1c628ce301b552cf3 > lens-cube/src/main/java/org/apache/lens/cube/parse/JoinResolver.java > 0bcc55855e11d2ce55ff73ee10674f9664dcc484 > > lens-cube/src/main/java/org/apache/lens/cube/parse/LightestDimensionResolver.java > 6074952ec2d2d49d24dd227186f4213ddba2ad86 > lens-cube/src/main/java/org/apache/lens/cube/parse/PruneCauses.java > PRE-CREATION > > lens-cube/src/main/java/org/apache/lens/cube/parse/StorageTableResolver.java > 06550b81988aa5cee06c86c9725770cadee58b3e > lens-cube/src/main/java/org/apache/lens/driver/cube/RewriteUtil.java > d2da97d3ad8d20690ff0d2e03d0ff860f6b70c90 > > Diff: https://reviews.apache.org/r/29919/diff/ > > > Testing > ------- > > error json looks like this now: > > ``` > { > "summary2,testfact2_raw,summary3": { > "cause": "INVALID_DENORM_TABLE" > }, > "summary4": { > "cause": "NO_CANDIDATE_STORAGES", > "storageCauses": { > "C2": "UNSUPPORTED" > }, > "updatePeriodCauses": {} > }, > "summary1,cheapfact,testfactmonthly,testfact2,testfact": { > "cause": "COLUMN_NOT_FOUND" > } > } > ``` > > > Thanks, > > Rajat Khandelwal > >
