I think we should move. JDK7 has been EOL for a couple years now and it's a pain to even install an old JDK7 these days.
rb On Thu, Jan 18, 2018 at 2:31 AM, Gabor Szadovszky < [email protected]> wrote: > Hi, > > The last commit in parquet-mr master > (c6764c4a0848abf1d581e22df8b33e28ee9f2ced) > does not build with jdk7 only with jdk8. We did not catch the issue because > either Travis and me use jdk8 to build parquet-mr. (The source level in the > pom.xml is set to 1.7 so both jdk7 and jdk8 should be able to build it but > jdk7 fails with “invalid inferred types for T; inferred type does not > conform to declared bound(s)”.) > > I think we have 2 options: > 1. Fix the code so it compiles with both jdk7 and jdk8. In this case > Travis should be updated to check both otherwise we might have similar > issues in the future. > 2. Bump up source/target levels to java8. > > What do you think? > > Thanks, > Gabor -- Ryan Blue Software Engineer Netflix
