Thanks Fokko.
I like the idea. Started a new "Dropping JDK 8 support" thread to ensure
transparency.

Best
Piotr


On Mon, 22 Jul 2024 at 15:24, Fokko Driesprong <fo...@apache.org> wrote:

> Thanks for summarizing this, Piotr.
>
> I believe having a separate thread on dropping Java 8 is the right thing
> to do. We want to be as transparent about these changes as possible.
>
> Kind regards,
> Fokko Driesprong
>
> Op ma 22 jul 2024 om 14:37 schreef Piotr Findeisen <
> piotr.findei...@gmail.com>:
>
>> Thanks for this lively discussion, it is great to see so many great
>> people involved!
>>
>> We have unanimous agreement that we add support for JDK 21.
>> Partial support (without spotless) will be added after 1.6.0 release is
>> out (just not to mess up with the release).
>> Full support (with spotless) will be added as soon as JDK 8 is dropped.
>> For 21 we have clarity and we don't have preconditions, this is great.
>> And this is a non-destructive operation too.
>>
>> We do not have agreement for dropping Hive module. This will be discussed
>> separately on a new thread.
>>
>> We also seem to have unanimous agreement for dropping JDK 8.
>> As to the timeline, it was proposed to do this in 2.0 release, so let's
>> roll with this, unless there are new objections
>> Since dropping support for something can be seen as a destructive
>> operation, does it require a formal vote?
>> Or do we treat +1 and -1 on this thread as votes already cast?
>>
>> Best,
>> Piotr
>>
>>
>>
>>
>>
>>
>> On Sat, 20 Jul 2024 at 00:12, Jack Ye <yezhao...@gmail.com> wrote:
>>
>>> +1 for dropping JDK8 support and adding JDK21.
>>>
>>> > What does dropping Java 8 support mean to companies that are still
>>> using Java 8 for Iceberg in production?
>>>
>>> From the AWS side, AWS Corretto JDK8 end of life is July 2026, see:
>>> https://aws.amazon.com/corretto/faqs/#support_calendar. I would suggest
>>> at least migrate before that time.
>>>
>>> -Jack
>>>
>>>
>>>
>>> On Fri, Jul 19, 2024 at 3:02 PM John Zhuge <jzh...@apache.org> wrote:
>>>
>>>> +1 adding java 21 support
>>>> +1 removing java 8 support
>>>>
>>>> On Fri, Jul 19, 2024 at 1:33 PM Daniel Weeks <dwe...@apache.org> wrote:
>>>>
>>>>> I'm also in favor of removing Java 8 support.  Hive docs state Hive 3
>>>>> requires java 8
>>>>> <https://github.com/apache/hive?tab=readme-ov-file#java> and in prior
>>>>> cases there were potential correctness issues when running with newer Java
>>>>> versions (these may have been addressed).
>>>>>
>>>>> As long as we're not updating the target version, I think we should be
>>>>> ok as they can still run in Java 8 if that remains a requirement.
>>>>>
>>>>> +1 to removing Java 8 support
>>>>> +1 to adding Java 21 support.
>>>>>
>>>>> -Dan
>>>>>
>>>>>
>>>>>
>>>>> On Fri, Jul 19, 2024 at 1:04 PM Ryan Blue <b...@databricks.com.invalid>
>>>>> wrote:
>>>>>
>>>>>> I agree that if we can separate the discussion about how to support
>>>>>> Hive, then we should do that.
>>>>>>
>>>>>> +1 to removing Java 8 support
>>>>>> +1 to adding Java 21 support.
>>>>>>
>>>>>> On Fri, Jul 19, 2024 at 12:58 PM huaxin gao <huaxin.ga...@gmail.com>
>>>>>> wrote:
>>>>>>
>>>>>>> +1 in favor of adding java 21 support
>>>>>>> +1 in favor of removing java 8 support
>>>>>>>
>>>>>>> I am currently working on Spark 4.0 / Iceberg integration
>>>>>>> <https://github.com/apache/iceberg/pull/10622>. Spark 4.0 runs on
>>>>>>> Java 17/21.
>>>>>>>
>>>>>>> On Fri, Jul 19, 2024 at 4:58 AM Piotr Findeisen <
>>>>>>> piotr.findei...@gmail.com> wrote:
>>>>>>>
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>> We recently started to test Hive3 with Java 11 and 17
>>>>>>>> <https://github.com/apache/iceberg/pull/10482> and the tests pass.
>>>>>>>> So dropping Java 8 doesn't technically require removing the Hive 3
>>>>>>>> related modules, unless users cannot do anything useful with them 
>>>>>>>> (because
>>>>>>>> e.g. they can only run Hive runtime with Java 8 for some reason).
>>>>>>>> Peter, can you please confirm this is not the case?
>>>>>>>> Then it seems we could proceed with JDK 8 drop and discuss what to
>>>>>>>> do with Hive modules *separately*.
>>>>>>>>
>>>>>>>> re original question of adding JDK 21 support -- we seem to have
>>>>>>>> strong consensus to add it.
>>>>>>>> Eduard plans to merge the PR once 1.6.0 is out. So I think we no
>>>>>>>> longer need to debate this topic, unless there are any new objections 
>>>>>>>> to be
>>>>>>>> raised.
>>>>>>>>
>>>>>>>>
>>>>>>>> Best
>>>>>>>> Piotr
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> On Fri, 19 Jul 2024 at 13:49, Péter Váry <
>>>>>>>> peter.vary.apa...@gmail.com> wrote:
>>>>>>>>
>>>>>>>>> Back to the main topic:
>>>>>>>>>
>>>>>>>>> Removing java8 support still allows us to use the old HMS
>>>>>>>>> libraries, so iceberg-hive-metastore module is not affected in this 
>>>>>>>>> sense.
>>>>>>>>> We still need to run the tests, to ensure that there is no changes in 
>>>>>>>>> the
>>>>>>>>> behaviour, but it is unlikely.
>>>>>>>>>
>>>>>>>>> +1 in favor of adding java 21 support
>>>>>>>>> +1 in favor of removing java 8 support - especially if it is not
>>>>>>>>> bound to removing Hive support as well
>>>>>>>>>
>>>>>>>>> Thanks,
>>>>>>>>> Peter
>>>>>>>>>
>>>>>>>>> Cheng Pan <pan3...@gmail.com> ezt írta (időpont: 2024. júl. 19.,
>>>>>>>>> P, 4:56):
>>>>>>>>>
>>>>>>>>>> A basic question, is iceberg-hive-metastore considered part of the
>>>>>>>>>> "Hive module"?
>>>>>>>>>>
>>>>>>>>>> I suppose that HMS 2.x is still widely used. AFAIK, the current
>>>>>>>>>> iceberg-hive-metastore is compatible with HMS 2.1+, based on
>>>>>>>>>> Iceberg
>>>>>>>>>> and Spark CI, I also suppose it works well with Java 8 to 21.
>>>>>>>>>>
>>>>>>>>>> Thanks,
>>>>>>>>>> Cheng Pan
>>>>>>>>>>
>>>>>>>>>> On Fri, Jul 19, 2024 at 4:44 AM Ryan Blue
>>>>>>>>>> <b...@databricks.com.invalid> wrote:
>>>>>>>>>> >
>>>>>>>>>> > Thanks for the context, Denys and Peter. Sounds like there's a
>>>>>>>>>> good question here about where the Hive integration should live and 
>>>>>>>>>> the
>>>>>>>>>> most recent decision was to maintain that support in Hive. I 
>>>>>>>>>> definitely
>>>>>>>>>> hear the point about Hive 3 users depending on the Iceberg modules. 
>>>>>>>>>> I'm
>>>>>>>>>> also glad to hear that some of the issues are expected to be fixed 
>>>>>>>>>> with the
>>>>>>>>>> release of Hive 4.0.x.
>>>>>>>>>> >
>>>>>>>>>> > I think that we have two separate questions for how to move
>>>>>>>>>> forward with Hive support, depending on the Hive version. There is a
>>>>>>>>>> question about what we do with the current Hive modules and what to 
>>>>>>>>>> do with
>>>>>>>>>> the Hive 4 support that has been developed externally.
>>>>>>>>>> >
>>>>>>>>>> > For Hive 2.x and 3.x, we have code in the Iceberg repo that is
>>>>>>>>>> not being developed. Hive 2 is fairly easy since it is EOL. While 
>>>>>>>>>> Hive 3 is
>>>>>>>>>> still used, I don't think it makes sense to keep releasing versions 
>>>>>>>>>> of it
>>>>>>>>>> if it requires Java 8, which has not been publicly maintained for 5 
>>>>>>>>>> years.
>>>>>>>>>> We need to upgrade and that is at odds with keeping support for Hive 
>>>>>>>>>> 3. As
>>>>>>>>>> Fokko and I both pointed out, people can still use older releases.
>>>>>>>>>> >
>>>>>>>>>> > For the question of how to maintain support for Hive 4, I think
>>>>>>>>>> it's worth having a separate discussion (probably not on the thread 
>>>>>>>>>> about
>>>>>>>>>> JDK versions) about where to maintain it. I think that it is best to
>>>>>>>>>> maintain integration in engines and not in the Iceberg project; 
>>>>>>>>>> there are
>>>>>>>>>> few implementations here and I think that it is a hard problem for 
>>>>>>>>>> Iceberg
>>>>>>>>>> to maintain support for multiple versions (as you can see with 
>>>>>>>>>> support for
>>>>>>>>>> so many different Flink, Hive, and Spark versions).
>>>>>>>>>> >
>>>>>>>>>> > Ryan
>>>>>>>>>> >
>>>>>>>>>> > On Thu, Jul 18, 2024 at 7:25 AM Denys Kuzmenko <
>>>>>>>>>> dkuzme...@apache.org> wrote:
>>>>>>>>>> >>
>>>>>>>>>> >> In the following 1-2 months we plan to release HIVE-4.0.1
>>>>>>>>>> which includes bug fixes and then focus on HIVE-4.1.0 release with 
>>>>>>>>>> jdk17.
>>>>>>>>>> >
>>>>>>>>>> >
>>>>>>>>>> >
>>>>>>>>>> > --
>>>>>>>>>> > Ryan Blue
>>>>>>>>>> > Databricks
>>>>>>>>>>
>>>>>>>>>
>>>>>>
>>>>>> --
>>>>>> Ryan Blue
>>>>>> Databricks
>>>>>>
>>>>>
>>>>
>>>> --
>>>> John Zhuge
>>>>
>>>

Reply via email to