----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54057/#review161691 -----------------------------------------------------------
Ship it! Ship It! - Puneet Gupta On Nov. 25, 2016, 6:16 a.m., Amareshwari Sriramadasu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54057/ > ----------------------------------------------------------- > > (Updated Nov. 25, 2016, 6:16 a.m.) > > > Review request for lens. > > > Bugs: LENS-1378 > https://issues.apache.org/jira/browse/LENS-1378 > > > Repository: lens > > > Description > ------- > > Fix is to add DenormalizationContext for each expression instead of being > part of main CubeQueryContext > > > Diffs > ----- > > > lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTableResolver.java > e7fc557 > lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryContext.java > e83ae76 > > lens-cube/src/main/java/org/apache/lens/cube/parse/DenormalizationResolver.java > 40ed387 > lens-cube/src/main/java/org/apache/lens/cube/parse/ExpressionResolver.java > 60dacdb > > lens-cube/src/main/java/org/apache/lens/cube/parse/QueriedPhraseContext.java > 11eb8f7 > lens-cube/src/main/java/org/apache/lens/cube/parse/TimeRangeChecker.java > 89b50f5 > lens-cube/src/main/java/org/apache/lens/cube/parse/TrackDenormContext.java > PRE-CREATION > lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java > 41ea83d > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestExpressionResolver.java > f2bb485 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestJoinResolver.java > 6430ed1 > tools/conf/server/logback.xml 63ab23c > > Diff: https://reviews.apache.org/r/54057/diff/ > > > Testing > ------- > > All cube tests passed > > > Thanks, > > Amareshwari Sriramadasu > >