[DISCUSS] Change default planner to blink planner in 1.11

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

[DISCUSS] Change default planner to blink planner in 1.11

Kurt Young
Hi Dev and User,

Blink planner for Table API & SQL is introduced in Flink 1.9 and already be the default planner for
SQL client in Flink 1.10. And since we already decided not introducing any new features to the 
original Flink planner, it already lacked of so many great features that the community has been working on, such as brand new type system, more DDL support and more planner capabilities. 
During this time, we've also received lots of great feedback from users who were trying to use blink
planner, both positive and negative (like bugs). This is a good sign, it at least shows more and more
users are starting to try out

So I want to start this discussion more formally to talk about replacing the default planner to blink.
Specifically, I want to gather feedbacks from dev and user about whether blink planner already 
cover the original planner's capabilities, what kind of issues you've ran into when try out blink 
planner and then make you fallback to original one. Since there is still a month to go when feature
freeze, there's still enough time for community to further enhance blink planner for this purpose. 

Let me know what you think, especially if you want to report or complain about something. Thanks 
in advance. 

Best,
Kurt
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Change default planner to blink planner in 1.11

Benchao Li
Hi Kurt,

It's excited to hear that the community aims to make Blink Planner default in 1.11. 

We have been using blink planner since 1.9 for streaming processing, it works very well, 
and covers many use cases in our company. 
So +1 to make it default in 1.11 from our side.

Kurt Young <[hidden email]> 于2020年4月1日周三 上午9:15写道:
Hi Dev and User,

Blink planner for Table API & SQL is introduced in Flink 1.9 and already be the default planner for
SQL client in Flink 1.10. And since we already decided not introducing any new features to the 
original Flink planner, it already lacked of so many great features that the community has been working on, such as brand new type system, more DDL support and more planner capabilities. 
During this time, we've also received lots of great feedback from users who were trying to use blink
planner, both positive and negative (like bugs). This is a good sign, it at least shows more and more
users are starting to try out

So I want to start this discussion more formally to talk about replacing the default planner to blink.
Specifically, I want to gather feedbacks from dev and user about whether blink planner already 
cover the original planner's capabilities, what kind of issues you've ran into when try out blink 
planner and then make you fallback to original one. Since there is still a month to go when feature
freeze, there's still enough time for community to further enhance blink planner for this purpose. 

Let me know what you think, especially if you want to report or complain about something. Thanks 
in advance. 

Best,
Kurt


--
Benchao Li
School of Electronics Engineering and Computer Science, Peking University
Tel:+86-15650713730
Email: [hidden email]; [hidden email]
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Change default planner to blink planner in 1.11

Jark Wu-3
+1 to make blink planner as default planner. 

We should give blink planner more exposure to encourage users trying out new features and lead users to migrate to blink planner. 

Glad to see blink planner is used in production since 1.9! @Benchao

Best,
Jark

On Wed, 1 Apr 2020 at 11:31, Benchao Li <[hidden email]> wrote:
Hi Kurt,

It's excited to hear that the community aims to make Blink Planner default in 1.11. 

We have been using blink planner since 1.9 for streaming processing, it works very well, 
and covers many use cases in our company. 
So +1 to make it default in 1.11 from our side.

Kurt Young <[hidden email]> 于2020年4月1日周三 上午9:15写道:
Hi Dev and User,

Blink planner for Table API & SQL is introduced in Flink 1.9 and already be the default planner for
SQL client in Flink 1.10. And since we already decided not introducing any new features to the 
original Flink planner, it already lacked of so many great features that the community has been working on, such as brand new type system, more DDL support and more planner capabilities. 
During this time, we've also received lots of great feedback from users who were trying to use blink
planner, both positive and negative (like bugs). This is a good sign, it at least shows more and more
users are starting to try out

So I want to start this discussion more formally to talk about replacing the default planner to blink.
Specifically, I want to gather feedbacks from dev and user about whether blink planner already 
cover the original planner's capabilities, what kind of issues you've ran into when try out blink 
planner and then make you fallback to original one. Since there is still a month to go when feature
freeze, there's still enough time for community to further enhance blink planner for this purpose. 

Let me know what you think, especially if you want to report or complain about something. Thanks 
in advance. 

Best,
Kurt


--
Benchao Li
School of Electronics Engineering and Computer Science, Peking University
Tel:+86-15650713730
Email: [hidden email]; [hidden email]
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Change default planner to blink planner in 1.11

Jingsong Li
+1

In 1.10, we have set default planner for SQL Client to Blink planner[1]. Looks good.


Best,
Jingsong Lee

On Wed, Apr 1, 2020 at 11:39 AM Jark Wu <[hidden email]> wrote:
+1 to make blink planner as default planner.

We should give blink planner more exposure to encourage users trying out
new features and lead users to migrate to blink planner.

Glad to see blink planner is used in production since 1.9! @Benchao

Best,
Jark

On Wed, 1 Apr 2020 at 11:31, Benchao Li <[hidden email]> wrote:

> Hi Kurt,
>
> It's excited to hear that the community aims to make Blink Planner default
> in 1.11.
>
> We have been using blink planner since 1.9 for streaming processing, it
> works very well,
> and covers many use cases in our company.
> So +1 to make it default in 1.11 from our side.
>
> Kurt Young <[hidden email]> 于2020年4月1日周三 上午9:15写道:
>
>> Hi Dev and User,
>>
>> Blink planner for Table API & SQL is introduced in Flink 1.9 and already
>> be the default planner for
>> SQL client in Flink 1.10. And since we already decided not introducing
>> any new features to the
>> original Flink planner, it already lacked of so many great features that
>> the community has been working on, such as brand new type system, more DDL
>> support and more planner capabilities.
>> During this time, we've also received lots of great feedback from users
>> who were trying to use blink
>> planner, both positive and negative (like bugs). This is a good sign, it
>> at least shows more and more
>> users are starting to try out.
>>
>> So I want to start this discussion more formally to talk about
>> replacing the default planner to blink.
>> Specifically, I want to gather feedbacks from dev and user about whether
>> blink planner already
>> cover the original planner's capabilities, what kind of issues you've ran
>> into when try out blink
>> planner and then make you fallback to original one. Since there is still
>> a month to go when feature
>> freeze, there's still enough time for community to further enhance blink
>> planner for this purpose.
>>
>> Let me know what you think, especially if you want to report or complain
>> about something. Thanks
>> in advance.
>>
>> Best,
>> Kurt
>>
>
>
> --
>
> Benchao Li
> School of Electronics Engineering and Computer Science, Peking University
> Tel:+86-15650713730
> Email: [hidden email]; [hidden email]
>
>


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

Re: [DISCUSS] Change default planner to blink planner in 1.11

Aljoscha Krettek
In reply to this post by Kurt Young
+1 to making Blink the default planner, we definitely don't want to
maintain two planners for much longer.

Best,
Aljoscha