> On Feb. 15, 2017, 9 a.m., Puneet Gupta wrote: > > lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java, line > > 2713 > > <https://reviews.apache.org/r/56649/diff/1/?file=1633075#file1633075line2713> > > > > Is this chnage required for java 8 ?
No. I was seeing some setup issue on my local machine in cube tests. This seemed to fix it. But this causes failure in two other tests. Hence, reverted. - Rajat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56649/#review165639 ----------------------------------------------------------- On Feb. 15, 2017, 11:24 a.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56649/ > ----------------------------------------------------------- > > (Updated Feb. 15, 2017, 11:24 a.m.) > > > Review request for lens. > > > Bugs: LENS-1387 > https://issues.apache.org/jira/browse/LENS-1387 > > > Repository: lens > > > Description > ------- > > Deprecating java7 and restricting lens to only compile with java 8 + > > > Diffs > ----- > > pom.xml 8ea64b77fff11d9f46be56437aed3b527bde0ff8 > > Diff: https://reviews.apache.org/r/56649/diff/ > > > Testing > ------- > > > Thanks, > > Rajat Khandelwal > >
