I agree with Péter, that sounds like the right approach to me as well. Kind regards, Fokko
Op vr 22 nov 2024 om 07:38 schreef Péter Váry <peter.vary.apa...@gmail.com>: > I would prefer B, and only revert to A if we find that B becomes too > complicated. > > On Fri, Nov 22, 2024, 04:26 Manu Zhang <owenzhang1...@gmail.com> wrote: > >> Hi Peter, >> >> Would you be more specific on which option above do you prefer? >> >> Thanks, >> Manu >> >> On Thu, Nov 21, 2024 at 10:07 PM Péter Váry <peter.vary.apa...@gmail.com> >> wrote: >> >>> Hi Team, >>> >>> Just to clarify. Hive 3 officially doesn't support Java 11, and there >>> are no plans to release a new Hive 3 version with support. >>> By "accident" the Hive Metastore tests are running with Hive 3 with Java >>> 11, but the Hive runtime tests are not running (Starting the HiveServer >>> fails, so no tests are running) >>> Currently we don't know how Hive 4 is working from the Iceberg repo (we >>> know that the Hive community is using Iceberg 1.6.1, so this shouldn't be a >>> big issue) >>> >>> Since Hive 3 is not officially supported, I also suggest moving forward, >>> and start using Hive 4. But we need to run our tests with Hive 4 first >>> before we change the documentation. >>> >>> Thanks, >>> Peter >>> >>> Jean-Baptiste Onofré <j...@nanthrax.net> ezt írta (időpont: 2024. nov. >>> 21., Cs, 14:21): >>> >>>> Hi Manu >>>> >>>> It sounds like a plan. I think it makes sense to drop Hive 2 & 3 and >>>> encourage use of Hive 4 (mostly documentation task). >>>> >>>> Regards >>>> JB >>>> >>>> On Wed, Nov 20, 2024 at 7:19 AM Manu Zhang <owenzhang1...@gmail.com> >>>> wrote: >>>> > >>>> > Okay, let me add this option >>>> > >>>> > D. Drop Hive 2 & 3 support and suggest to use built-in Iceberg >>>> support of Hive 4 >>>> > >>>> > On Wed, Nov 20, 2024 at 2:00 PM Cheng Pan <pan3...@gmail.com> wrote: >>>> >> >>>> >> Hive 4 brings built-in support for Iceberg format, duplicated >>>> implementation in both sides look a redundant stuff. >>>> >> >>>> >> As Hive 2 and 3 do not support Java 11+, and Iceberg 1.8 requires >>>> Java 11+, the combination is invalid. How about simply dropping support for >>>> Hive 2&3 and suggesting the Hive user upgrade Hive 4 to gain the built-in >>>> Iceberg support? >>>> >> >>>> >> Thanks, >>>> >> Cheng Pan >>>> >> >>>> >> >>>> >> >>>> >> On Nov 20, 2024, at 12:47, Manu Zhang <owenzhang1...@gmail.com> >>>> wrote: >>>> >> >>>> >> Hi all, >>>> >> >>>> >> We previously reached consensus[1] to deprecate Hive 2 in 1.7 and >>>> drop in 1.8. However, when working on the removal PR[2], multiple tests >>>> failed in Hive 3 due to not supporting JDK11[3]. The fix has been >>>> back-ported to branch-3.1[4] but not released yet. As announced on Hive >>>> website, Hive 3.x is declared as End of Life so there will be no more Hive >>>> 3 release. Peter(@pvary) suggested upgrading to Hive 4 instead. On the >>>> other hand, iceberg-hive3 tests are already broken after we dropped JDK 8 >>>> support. It's not caught previously due to tests not running[6]. >>>> >> >>>> >> Based on the current situation, here are the options I can think of >>>> to move forward >>>> >> >>>> >> A. Continue to remove Hive 2 in the current PR and upgrade to Hive 4 >>>> in a separate PR. >>>> >> B. Hold on removing Hive 2 until we upgrade to Hive 4 >>>> >> C. Add source dependency[7] on Hive branch-3.1 or make a Hive 3.1 >>>> release from a forked repo. >>>> >> >>>> >> 1. https://lists.apache.org/thread/zg14b8cor4lnbyd3t4n1297y2bwb1fsg >>>> >> 2. https://github.com/apache/iceberg/pull/10996 >>>> >> 3. https://issues.apache.org/jira/browse/HIVE-21584 >>>> >> 4. https://github.com/apache/hive/commits/branch-3.1/ >>>> >> 5. https://hive.apache.org/general/downloads/ >>>> >> 6. https://github.com/apache/iceberg/pull/11584 >>>> >> 7. https://blog.gradle.org/introducing-source-dependencies >>>> >> >>>> >> Which option do you prefer? Any better alternative? >>>> >> >>>> >> Thanks, >>>> >> Manu >>>> >> >>>> >> >>>> >>>