[DISCUSS] Move flink-orc to flink-formats from flink-connectors

classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|

[DISCUSS] Move flink-orc to flink-formats from flink-connectors

Jingsong Lee
Hi devs,

I found that Orc is still in connectors, but we already have the parent
model of formats.
- Is it better to put flink-orc in flink-formats?
- Is there a compatibility issue with the move? Looks like only the parent
model needs to be changed, and no other changes are needed.

--
Best, Jingsong Lee
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-orc to flink-formats from flink-connectors

Zhenghua Gao
+1 to move flink-orc to flink-formats.
Since we have put other file-based formats(flink-avro, flink-json,
flink-parquet, flink-json, flink-csv, flink-sequence-file)
in flink-formats, flink-orc should be the same.

*Best Regards,*
*Zhenghua Gao*


On Tue, Oct 29, 2019 at 11:10 AM Jingsong Lee <[hidden email]>
wrote:

> Hi devs,
>
> I found that Orc is still in connectors, but we already have the parent
> model of formats.
> - Is it better to put flink-orc in flink-formats?
> - Is there a compatibility issue with the move? Looks like only the parent
> model needs to be changed, and no other changes are needed.
>
> --
> Best, Jingsong Lee
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-orc to flink-formats from flink-connectors

bowen.li
Hi Jingsong,

Thanks for bringing it up. I found flink-orc lives under flink-connectors
instead of flink-formats to be confusing too.

I wonder why it's that way in the beginning? If there's no good reason to
remain that way, +1 to move flink-orc to flink-formats

Bowen


On Mon, Oct 28, 2019 at 11:26 PM Zhenghua Gao <[hidden email]> wrote:

> +1 to move flink-orc to flink-formats.
> Since we have put other file-based formats(flink-avro, flink-json,
> flink-parquet, flink-json, flink-csv, flink-sequence-file)
> in flink-formats, flink-orc should be the same.
>
> *Best Regards,*
> *Zhenghua Gao*
>
>
> On Tue, Oct 29, 2019 at 11:10 AM Jingsong Lee <[hidden email]>
> wrote:
>
> > Hi devs,
> >
> > I found that Orc is still in connectors, but we already have the parent
> > model of formats.
> > - Is it better to put flink-orc in flink-formats?
> > - Is there a compatibility issue with the move? Looks like only the
> parent
> > model needs to be changed, and no other changes are needed.
> >
> > --
> > Best, Jingsong Lee
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-orc to flink-formats from flink-connectors

Jark Wu-2
+1 to move to flink-format.

Best,
Jark

On Wed, 30 Oct 2019 at 02:27, Bowen Li <[hidden email]> wrote:

> Hi Jingsong,
>
> Thanks for bringing it up. I found flink-orc lives under flink-connectors
> instead of flink-formats to be confusing too.
>
> I wonder why it's that way in the beginning? If there's no good reason to
> remain that way, +1 to move flink-orc to flink-formats
>
> Bowen
>
>
> On Mon, Oct 28, 2019 at 11:26 PM Zhenghua Gao <[hidden email]> wrote:
>
> > +1 to move flink-orc to flink-formats.
> > Since we have put other file-based formats(flink-avro, flink-json,
> > flink-parquet, flink-json, flink-csv, flink-sequence-file)
> > in flink-formats, flink-orc should be the same.
> >
> > *Best Regards,*
> > *Zhenghua Gao*
> >
> >
> > On Tue, Oct 29, 2019 at 11:10 AM Jingsong Lee <[hidden email]>
> > wrote:
> >
> > > Hi devs,
> > >
> > > I found that Orc is still in connectors, but we already have the parent
> > > model of formats.
> > > - Is it better to put flink-orc in flink-formats?
> > > - Is there a compatibility issue with the move? Looks like only the
> > parent
> > > model needs to be changed, and no other changes are needed.
> > >
> > > --
> > > Best, Jingsong Lee
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-orc to flink-formats from flink-connectors

Danny Chan
In reply to this post by Jingsong Lee
+1 to move to flink-format.

Best,
Danny Chan
在 2019年10月29日 +0800 AM11:10,[hidden email],写道:
>
> +1 to move to flink-format.
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-orc to flink-formats from flink-connectors

Jingsong Li
Hi bowen,
I think the reason is that flink-orc was developed earlier than
flink-formats.

Thanks the responses of everyone, I will create JIRA to do it.

On Wed, Oct 30, 2019 at 10:27 AM Danny Chan <[hidden email]> wrote:

> +1 to move to flink-format.
>
> Best,
> Danny Chan
> 在 2019年10月29日 +0800 AM11:10,[hidden email],写道:
> >
> > +1 to move to flink-format.
>


--
Best, Jingsong Lee
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-orc to flink-formats from flink-connectors

Jark Wu-2
Hi Jingsong,

Do we need a formal vote? Or do we need to wait for "Lazy
Majority" approval [1] which requires 3 binding votes?

Best,
Jark

[1]:
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=120731026

On Wed, 30 Oct 2019 at 11:12, Jingsong Li <[hidden email]> wrote:

> Hi bowen,
> I think the reason is that flink-orc was developed earlier than
> flink-formats.
>
> Thanks the responses of everyone, I will create JIRA to do it.
>
> On Wed, Oct 30, 2019 at 10:27 AM Danny Chan <[hidden email]> wrote:
>
> > +1 to move to flink-format.
> >
> > Best,
> > Danny Chan
> > 在 2019年10月29日 +0800 AM11:10,[hidden email],写道:
> > >
> > > +1 to move to flink-format.
> >
>
>
> --
> Best, Jingsong Lee
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-orc to flink-formats from flink-connectors

Jingsong Li
Thanks jark for reminding me.
I will start a vote thread.

On Wed, Oct 30, 2019 at 11:40 AM Jark Wu <[hidden email]> wrote:

> Hi Jingsong,
>
> Do we need a formal vote? Or do we need to wait for "Lazy
> Majority" approval [1] which requires 3 binding votes?
>
> Best,
> Jark
>
> [1]:
> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=120731026
>
> On Wed, 30 Oct 2019 at 11:12, Jingsong Li <[hidden email]> wrote:
>
> > Hi bowen,
> > I think the reason is that flink-orc was developed earlier than
> > flink-formats.
> >
> > Thanks the responses of everyone, I will create JIRA to do it.
> >
> > On Wed, Oct 30, 2019 at 10:27 AM Danny Chan <[hidden email]>
> wrote:
> >
> > > +1 to move to flink-format.
> > >
> > > Best,
> > > Danny Chan
> > > 在 2019年10月29日 +0800 AM11:10,[hidden email],写道:
> > > >
> > > > +1 to move to flink-format.
> > >
> >
> >
> > --
> > Best, Jingsong Lee
> >
>


--
Best, Jingsong Lee
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-orc to flink-formats from flink-connectors

Fabian Hueske-2
+1 from my side as well.

Am Mi., 30. Okt. 2019 um 04:51 Uhr schrieb Jingsong Li <
[hidden email]>:

> Thanks jark for reminding me.
> I will start a vote thread.
>
> On Wed, Oct 30, 2019 at 11:40 AM Jark Wu <[hidden email]> wrote:
>
> > Hi Jingsong,
> >
> > Do we need a formal vote? Or do we need to wait for "Lazy
> > Majority" approval [1] which requires 3 binding votes?
> >
> > Best,
> > Jark
> >
> > [1]:
> >
> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=120731026
> >
> > On Wed, 30 Oct 2019 at 11:12, Jingsong Li <[hidden email]>
> wrote:
> >
> > > Hi bowen,
> > > I think the reason is that flink-orc was developed earlier than
> > > flink-formats.
> > >
> > > Thanks the responses of everyone, I will create JIRA to do it.
> > >
> > > On Wed, Oct 30, 2019 at 10:27 AM Danny Chan <[hidden email]>
> > wrote:
> > >
> > > > +1 to move to flink-format.
> > > >
> > > > Best,
> > > > Danny Chan
> > > > 在 2019年10月29日 +0800 AM11:10,[hidden email],写道:
> > > > >
> > > > > +1 to move to flink-format.
> > > >
> > >
> > >
> > > --
> > > Best, Jingsong Lee
> > >
> >
>
>
> --
> Best, Jingsong Lee
>