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 >>> >> >>> >> >>> >>