----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59803/#review176979 -----------------------------------------------------------
lens-cube/src/main/java/org/apache/lens/driver/cube/RewriterPlan.java Line 50 (original), 49 (patched) <https://reviews.apache.org/r/59803/#comment250545> Should we check !ctx.getQueryWriterContext().getDimsToQuery().isEmpty() and ctx.getQueryWriterContext().getDimsToQuery()!=null - Puneet Gupta On June 5, 2017, 2:18 p.m., Sushil Mohanty wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59803/ > ----------------------------------------------------------- > > (Updated June 5, 2017, 2:18 p.m.) > > > Review request for lens. > > > Bugs: LENS-1432 > https://issues.apache.org/jira/browse/LENS-1432 > > > Repository: lens > > > Description > ------- > > CubeQueryContext.getDimsToQuery() is empty, causing the problem. > > > Diffs > ----- > > > lens-cube/src/main/java/org/apache/lens/cube/parse/MultiCandidateQueryWriterContext.java > d57c027 > lens-cube/src/main/java/org/apache/lens/cube/parse/QueryWriterContext.java > d55de1f > lens-cube/src/main/java/org/apache/lens/driver/cube/RewriterPlan.java > caf8770 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestRewriterPlan.java > 76ea77d > > > Diff: https://reviews.apache.org/r/59803/diff/2/ > > > Testing > ------- > > > Thanks, > > Sushil Mohanty > >
