[DISCUSS] Move flink-python to opt

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

[DISCUSS] Move flink-python to opt

Ufuk Celebi-2
Dear devs,

what are your thoughts about moving flink-python (batch API) to opt
instead of having it in lib? The streaming counter part
(flink-streaming-python) is only as part of opt.

I think we don't have many users of the Python batch API. I think this
will make the streaming/batch experience more consistent and would
result in a cleaner default classpath.

Best,

Ufuk
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-python to opt

Jeff Zhang
+1

Ufuk Celebi <[hidden email]> 于2019年2月25日周一 下午10:33写道:

> Dear devs,
>
> what are your thoughts about moving flink-python (batch API) to opt
> instead of having it in lib? The streaming counter part
> (flink-streaming-python) is only as part of opt.
>
> I think we don't have many users of the Python batch API. I think this
> will make the streaming/batch experience more consistent and would
> result in a cleaner default classpath.
>
> Best,
>
> Ufuk
>


--
Best Regards

Jeff Zhang
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-python to opt

dwysakowicz
Hi Ufuk,

I think that makes absolute sense. +1 for this.

Best,

Dawid

On 25/02/2019 18:46, Jeff Zhang wrote:

> +1
>
> Ufuk Celebi <[hidden email]> 于2019年2月25日周一 下午10:33写道:
>
>> Dear devs,
>>
>> what are your thoughts about moving flink-python (batch API) to opt
>> instead of having it in lib? The streaming counter part
>> (flink-streaming-python) is only as part of opt.
>>
>> I think we don't have many users of the Python batch API. I think this
>> will make the streaming/batch experience more consistent and would
>> result in a cleaner default classpath.
>>
>> Best,
>>
>> Ufuk
>>
>


signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-python to opt

vino yang
+1

Best,
Vino

Dawid Wysakowicz <[hidden email]> 于2019年2月26日周二 上午3:07写道:

> Hi Ufuk,
>
> I think that makes absolute sense. +1 for this.
>
> Best,
>
> Dawid
>
> On 25/02/2019 18:46, Jeff Zhang wrote:
> > +1
> >
> > Ufuk Celebi <[hidden email]> 于2019年2月25日周一 下午10:33写道:
> >
> >> Dear devs,
> >>
> >> what are your thoughts about moving flink-python (batch API) to opt
> >> instead of having it in lib? The streaming counter part
> >> (flink-streaming-python) is only as part of opt.
> >>
> >> I think we don't have many users of the Python batch API. I think this
> >> will make the streaming/batch experience more consistent and would
> >> result in a cleaner default classpath.
> >>
> >> Best,
> >>
> >> Ufuk
> >>
> >
>
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-python to opt

jincheng sun
In reply to this post by Ufuk Celebi-2
Thanks for bringing up this DISCUSS Ufuk!

Makes sense.  +1 for doing this.

Best,
Jincheng

Ufuk Celebi <[hidden email]> 于2019年2月25日周一 下午10:33写道:

> Dear devs,
>
> what are your thoughts about moving flink-python (batch API) to opt
> instead of having it in lib? The streaming counter part
> (flink-streaming-python) is only as part of opt.
>
> I think we don't have many users of the Python batch API. I think this
> will make the streaming/batch experience more consistent and would
> result in a cleaner default classpath.
>
> Best,
>
> Ufuk
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-python to opt

Till Rohrmann
+1 from my side for the proposal.

Cheers,
Till

On Tue, Feb 26, 2019 at 7:37 AM jincheng sun <[hidden email]>
wrote:

> Thanks for bringing up this DISCUSS Ufuk!
>
> Makes sense.  +1 for doing this.
>
> Best,
> Jincheng
>
> Ufuk Celebi <[hidden email]> 于2019年2月25日周一 下午10:33写道:
>
> > Dear devs,
> >
> > what are your thoughts about moving flink-python (batch API) to opt
> > instead of having it in lib? The streaming counter part
> > (flink-streaming-python) is only as part of opt.
> >
> > I think we don't have many users of the Python batch API. I think this
> > will make the streaming/batch experience more consistent and would
> > result in a cleaner default classpath.
> >
> > Best,
> >
> > Ufuk
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Move flink-python to opt

Ufuk Celebi-4
Thanks for the feedback.

I created a PR in https://github.com/apache/flink/pull/7843.

@Aljoscha Krettek: Do you think we can include this in the 1.8 branch?

– Ufuk

On Tue, Feb 26, 2019 at 3:37 PM Till Rohrmann <[hidden email]> wrote:

>
> +1 from my side for the proposal.
>
> Cheers,
> Till
>
> On Tue, Feb 26, 2019 at 7:37 AM jincheng sun <[hidden email]>
> wrote:
>
> > Thanks for bringing up this DISCUSS Ufuk!
> >
> > Makes sense.  +1 for doing this.
> >
> > Best,
> > Jincheng
> >
> > Ufuk Celebi <[hidden email]> 于2019年2月25日周一 下午10:33写道:
> >
> > > Dear devs,
> > >
> > > what are your thoughts about moving flink-python (batch API) to opt
> > > instead of having it in lib? The streaming counter part
> > > (flink-streaming-python) is only as part of opt.
> > >
> > > I think we don't have many users of the Python batch API. I think this
> > > will make the streaming/batch experience more consistent and would
> > > result in a cleaner default classpath.
> > >
> > > Best,
> > >
> > > Ufuk
> > >
> >