> On March 23, 2015, 1:28 p.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java, > > line 118 > > <https://reviews.apache.org/r/32388/diff/1/?file=902607#file902607line118> > > > > brief error should be missing partitions
Yes working on that. > On March 23, 2015, 1:28 p.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTablePruneCause.java, > > line 99 > > <https://reviews.apache.org/r/32388/diff/1/?file=902605#file902605line99> > > > > We need to have this back. May be the storage cause should get bubbled > > up. We can keep both. > On March 23, 2015, 1:28 p.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java, > > line 869 > > <https://reviews.apache.org/r/32388/diff/1/?file=902607#file902607line869> > > > > No_candidate-storage cause is fully gone? Will put it back. Anyway cheapfact will always give no supported candidate storages. > On March 23, 2015, 1:28 p.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java, > > line 857 > > <https://reviews.apache.org/r/32388/diff/1/?file=902607#file902607line857> > > > > brief should be missing partitions Working on that. - Rajat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32388/#review77380 ----------------------------------------------------------- On March 23, 2015, 12:56 p.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32388/ > ----------------------------------------------------------- > > (Updated March 23, 2015, 12:56 p.m.) > > > Review request for lens. > > > Bugs: LENS-412 > https://issues.apache.org/jira/browse/LENS-412 > > > Repository: lens > > > Description > ------- > > some moving around in error codes. > > > Diffs > ----- > > > lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTablePruneCause.java > bfeff4f7443a3e8d0adcf04dc9a5a440e5a3fc24 > > lens-cube/src/main/java/org/apache/lens/cube/parse/StorageTableResolver.java > c29d88f1fe32c62947af6ca44179299d446c638c > lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java > 42782297118dbff231cc2a02de8813f58162b705 > > Diff: https://reviews.apache.org/r/32388/diff/ > > > Testing > ------- > > > Thanks, > > Rajat Khandelwal > >
