> Anton, do you have something in mind about Impala that you are interested in?

Was just curious about the plan for JDK versions overall. Thanks, Zoltan!

> On Apr 27, 2023, at 1:56 AM, Zoltán Borók-Nagy <borokna...@apache.org> wrote:
> 
> Impala works well with JDK11 with --release=8, so for us this would probably 
> be the best option for now.
> Hopefully we can also move to JDK11 soon (IMPALA-11360 
> <https://issues.apache.org/jira/browse/IMPALA-11360>).
> 
> Anton, do you have something in mind about Impala that you are interested in?
> 
> - Zoltan
> 
> 
> On Wed, Apr 26, 2023 at 10:48 PM Jack Ye <yezhao...@gmail.com 
> <mailto:yezhao...@gmail.com>> wrote:
> Added issue https://github.com/apache/iceberg/issues/7440 
> <https://github.com/apache/iceberg/issues/7440>
> 
> -Jack
> 
> On Wed, Apr 26, 2023 at 8:28 AM Anton Okolnychyi 
> <aokolnyc...@apple.com.invalid> wrote:
> Is anyone interested to give it a try?
> 
> - Anton
> 
>> On Apr 25, 2023, at 4:13 PM, Ryan Blue <b...@tabular.io 
>> <mailto:b...@tabular.io>> wrote:
>> 
>> Yeah, I do like the idea of using --release. We'll need to test it with 
>> those platforms though.
>> 
>> On Tue, Apr 25, 2023 at 12:04 PM Anton Okolnychyi 
>> <aokolnyc...@apple.com.invalid <mailto:aokolnyc...@apple.com.invalid>> wrote:
>> It would be interesting to hear more from Hive/Impala folks on this topic.
>> 
>> I’d avoid supporting both JDK8 and JDK11 given the complexity we need to 
>> add. Our build is already complicated.
>> 
>> Jack’s idea of using JDK11 with --release flag may be worth exploring.
>> 
>> - Anton
>> 
>>> On Apr 24, 2023, at 10:11 AM, Jack Ye <yezhao...@gmail.com 
>>> <mailto:yezhao...@gmail.com>> wrote:
>>> 
>>> I agree, it wouldn't help given the fact that we won't be able to use the 
>>> features in newer JDKs.
>>> 
>>> However, I think there is still a difference in the artifact compiled by 
>>> JDK8, vs compiled by JDK11 with --release=8, that might be useful. For 
>>> example, I came across this try with resource introduces unreachable byte 
>>> code issue 
>>> <https://stackoverflow.com/questions/25615417/try-with-resources-introduce-unreachable-bytecode/25746587#25746587>
>>>  which is fixed in JDK11, and some optimizations in JDK9 related to string 
>>> concatenation <https://openjdk.org/jeps/280>. Although Javac is not an 
>>> optimizing compiler and there should not be much difference in performance 
>>> of the jars produced by different compilers, these changes might be worth 
>>> for the project to declare a newer compile-time JDK across all modules, and 
>>> just always enforce --release flag to 8 to ensure people do not use newer 
>>> JDK features. But these are quite minor changes so I am not very 
>>> opinionated either way.
>>> 
>>> Any thoughts?
>>> 
>>> -Jack
>>> 
>>> On Mon, Apr 24, 2023 at 2:22 AM Zoltán Borók-Nagy <borokna...@apache.org 
>>> <mailto:borokna...@apache.org>> wrote:
>>> Besides Hive, neither Impala is compatible with Java11 right now. This work 
>>> is in-progress: https://issues.apache.org/jira/browse/IMPALA-11360 
>>> <https://issues.apache.org/jira/browse/IMPALA-11360>
>>> 
>>> - Zoltan
>>> 
>>> On Mon, Apr 24, 2023 at 11:07 AM Mass Dosage <massdos...@gmail.com 
>>> <mailto:massdos...@gmail.com>> wrote:
>>> I agree with Ryan, unless you can change the source version there's not 
>>> that much point. 
>>> 
>>> On the Hive front, as you can see from that ticket it's been open for 4(!) 
>>> years and hasn't received much action recently. I think it's one of the 
>>> reasons AWS EMR still defaults to Java 8. It would be really great if they 
>>> could finally push that one over the finish line.
>>> 
>>> On Sat, 22 Apr 2023 at 20:43, Ryan Blue <b...@tabular.io 
>>> <mailto:b...@tabular.io>> wrote:
>>> I think in order to publish multiple versions we would need to have 
>>> different artifact names, like Scala uses (e.g. _2.12).
>>> 
>>> It probably also wouldn't help. If we have to remain compatible with JDK 8, 
>>> then publishing some artifacts for JDK 11 would still mean only using JDK 8 
>>> features. The source version is what we care about more, so if we can't 
>>> change it then we can't really do anything else.
>>> 
>>> On Sat, Apr 22, 2023 at 10:12 AM Jack Ye <yezhao...@gmail.com 
>>> <mailto:yezhao...@gmail.com>> wrote:
>>> Would it be an option to use --release flag to control the release target 
>>> version, and publish 2 versions of the library to Maven, 1 for JDK8 and 1 
>>> for JDK11?
>>> 
>>> Jack
>>> 
>>> On Fri, Apr 21, 2023 at 5:17 PM Ryan Blue <b...@tabular.io 
>>> <mailto:b...@tabular.io>> wrote:
>>> Looks like Hive isn't quite done migrating to Java 11: 
>>> https://issues.apache.org/jira/browse/HIVE-22415 
>>> <https://issues.apache.org/jira/browse/HIVE-22415>
>>> 
>>> I'm not sure whether that's still a problem, but we currently don't build 
>>> Hive 3 support unless we're using Java 8. That makes me think that dropping 
>>> JDK 8 support would probably also make it a lot more difficult for Hive to 
>>> do releases based on Iceberg. Even with some of the integration moving into 
>>> the Hive project, if we started shipping JDK 11 Jars then Hive would no 
>>> longer be able to update.
>>> 
>>> Ryan
>>> 
>>> On Fri, Apr 21, 2023 at 5:02 PM Anton Okolnychyi 
>>> <aokolnyc...@apple.com.invalid <mailto:aokolnyc...@apple.com.invalid>> 
>>> wrote:
>>> Sorry, I wasn’t clear that I also imply dropping JDK 8 (unless there is a 
>>> good reason to keep it?).
>>> 
>>> - Anton
>>> 
>>>> On Apr 21, 2023, at 4:59 PM, Ryan Blue <b...@tabular.io 
>>>> <mailto:b...@tabular.io>> wrote:
>>>> 
>>>> Would we also drop support for JDK 8?
>>>> 
>>>> On Fri, Apr 21, 2023 at 4:58 PM Anton Okolnychyi 
>>>> <aokolnyc...@apple.com.invalid <mailto:aokolnyc...@apple.com.invalid>> 
>>>> wrote:
>>>> Following up on the discussion in the Spark 2.4 thread, shall we move to 
>>>> JDK 11 for releases as Spark 2.4 support has been dropped?
>>>> 
>>>> - Anton
>>>> 
>>>> 
>>>> -- 
>>>> Ryan Blue
>>>> Tabular
>>> 
>>> 
>>> 
>>> -- 
>>> Ryan Blue
>>> Tabular
>>> 
>>> 
>>> -- 
>>> Ryan Blue
>>> Tabular
>> 
>> 
>> 
>> -- 
>> Ryan Blue
>> Tabular
> 

Reply via email to