Re: [DISCUSS] Rename iceberg-hive module?

2020-08-19 Thread Jungtaek Lim
+1 for `iceberg-hive-metastore` and also +1 for RD's proposal. Thanks, Jungtaek Lim (HeartSaVioR) On Thu, Aug 20, 2020 at 11:20 AM Jingsong Li wrote: > +1 for `iceberg-hive-metastore` > > I'm confused about `iceberg-hive` and `iceberg-mr`. > > Best, > Jingsong > > On Thu, Aug 20, 2020 at

Re: [DISCUSS] Rename iceberg-hive module?

2020-08-19 Thread Jingsong Li
+1 for `iceberg-hive-metastore` I'm confused about `iceberg-hive` and `iceberg-mr`. Best, Jingsong On Thu, Aug 20, 2020 at 9:48 AM Dongjoon Hyun wrote: > +1 for `iceberg-hive-metastore`. > > Maybe, is `Apache Iceberg 1.0.0` a good candidate to have that breaking > change? > > Bests, >

Re: [DISCUSS] Rename iceberg-hive module?

2020-08-19 Thread Dongjoon Hyun
+1 for `iceberg-hive-metastore`. Maybe, is `Apache Iceberg 1.0.0` a good candidate to have that breaking change? Bests, Dongjoon. On Wed, Aug 19, 2020 at 6:35 PM RD wrote: > I'm +1 for this rename. I think we should keep the iceberg-mr module as > is and maybe add a new module

Re: [DISCUSS] Rename iceberg-hive module?

2020-08-19 Thread RD
I'm +1 for this rename. I think we should keep the iceberg-mr module as is and maybe add a new module iceberg-hive-exec [not sure if it is a good idea to salvage iceberg-hive for this purpose] which contains hive specific StorageHandler, Serde and IcebergHivInputFormat classes. -R On Wed, Aug

[DISCUSS] Rename iceberg-hive module?

2020-08-19 Thread Ryan Blue
In the discussion this morning, we talked about what to name the runtime module we want to add for Hive, iceberg-hive-runtime. Unfortunately, iceberg-hive is the Hive _metastore_ module, so it is a bit misleading to name the Hive runtime module iceberg-hive-runtime. It was also pointed out that

Re: [VOTE] Release Apache Iceberg 0.9.1 RC0

2020-08-19 Thread Jungtaek Lim
Just FYI, looks like the 0.9.1 artifacts are available now, but the release page on the website hasn't been updated yet. On Sat, Aug 15, 2020 at 9:46 AM Ryan Blue wrote: > With 8 +1 votes and no others, this RC passes. Thanks for validating the > patch release, everyone! > > I'll get started on