Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-08 Thread Jing Ge
an, mentioned in the FLIP that 1.18 > will > >> > > >> support Calc, HashJoin and HashAgg, then what will be the next > >> step? > >> > and > >> > > >> which operators do we ultimately expect to cover (all or specific > >

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-07 Thread Aitozi
, > > > HashJoin, > > > > >> HashAgg, etc. are more commonly used, so we will support them > first. > > > > Later > > > > >> we support the rest of the operators step by step. Considering the > > > time > > > > &g

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-07 Thread liu ron
ritize them according to their usage. Operators like Calc, >> > HashJoin, >> > > >> HashAgg, etc. are more commonly used, so we will support them >> first. >> > > Later >> > > >> we support the rest of the operators step by step. Considering the >>

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-07 Thread liu ron
; Ron > > > >> > > > >> Jingsong Li 于2023年6月5日周一 14:15写道: > > > >> > > > >>> > For the state compatibility session, it seems that the checkpoint > > > >>> compatibility would be broken just like [1] did. Could FLI

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-07 Thread Jing Ge
e > another > > >>> story? > > >>> > > >>> Best, > > >>> Jingsong > > >>> > > >>> On Mon, Jun 5, 2023 at 2:07 PM Yun Tang wrote: > > >>> > > > >>> > Hi Ron, > > >

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-07 Thread Aitozi
ce, looking > >>> forward to its completion in Flink! > >>> > > >>> > For the state compatibility session, it seems that the checkpoint > >>> compatibility would be broken just like [1] did. Could FLIP-190 [2] > still > >>&g

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-06 Thread liu ron
he state compatibility session, it seems that the checkpoint >>> compatibility would be broken just like [1] did. Could FLIP-190 [2] still >>> be helpful in this case for SQL version upgrades? >>> > >>> > >>> > [1] >>> https://docs.google.com/document/d/1qKVohV12

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-05 Thread liu ron
sI/edit#heading=h.fri5rtpte0si >> > [2] >> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=191336489 >> > >> > Best >> > Yun Tang >> > >> > >> > From: Lincoln Lee >> > Sent

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-05 Thread liu ron
t; > [2] > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=191336489 > > > > Best > > Yun Tang > > > > ________________ > > From: Lincoln Lee > > Sent: Monday, June 5, 2023 10:56 > > To: dev@flink.apache.org > > Subject: Re: [DISCUSS]

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-05 Thread Benchao Li
; [2] > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=191336489 > > > > Best > > Yun Tang > > > > ________________ > > From: Lincoln Lee > > Sent: Monday, June 5, 2023 10:56 > > To: dev@flink.apache.org > > Subject: Re: [DISCUSS] FLIP-315: Support O

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-05 Thread Jingsong Li
iewpage.action?pageId=191336489 > > Best > Yun Tang > > > From: Lincoln Lee > Sent: Monday, June 5, 2023 10:56 > To: dev@flink.apache.org > Subject: Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL > > Hi Ron > > OFGC lo

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-05 Thread Yun Tang
To: dev@flink.apache.org Subject: Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL Hi Ron OFGC looks like an exciting optimization, looking forward to its completion in Flink! A small question, do we consider adding a benchmark for the operators to intuitively understand

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-04 Thread Lincoln Lee
Hi Ron OFGC looks like an exciting optimization, looking forward to its completion in Flink! A small question, do we consider adding a benchmark for the operators to intuitively understand the improvement brought by each improvement? In addition, for the implementation plan, mentioned in the FLIP

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-04 Thread liu ron
Hi, Jark Thanks for your feedback, according to my initial assessment, the work effort is relatively large. Moreover, I will add a test result of all queries to the FLIP. Best, Ron Jark Wu 于2023年6月1日周四 20:45写道: > Hi Ron, > > Thanks a lot for the great proposal. The FLIP looks good to me in

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-06-01 Thread Jark Wu
Hi Ron, Thanks a lot for the great proposal. The FLIP looks good to me in general. It looks like not an easy work but the performance sounds promising. So I think it's worth doing. Besides, if there is a complete test graph with all TPC-DS queries, the effect of this FLIP will be more intuitive.

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-05-31 Thread liu ron
Hi, Jinsong Thanks for your valuable suggestions. Best, Ron Jingsong Li 于2023年5月30日周二 13:22写道: > Thanks Ron for your information. > > I suggest that it can be written in the Motivation of FLIP. > > Best, > Jingsong > > On Tue, May 30, 2023 at 9:57 AM liu ron wrote: > > > > Hi, Jingsong > > >

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-05-29 Thread Jingsong Li
Thanks Ron for your information. I suggest that it can be written in the Motivation of FLIP. Best, Jingsong On Tue, May 30, 2023 at 9:57 AM liu ron wrote: > > Hi, Jingsong > > Thanks for your review. We have tested it in TPC-DS case, and got a 12% > gain overall when only supporting only Calc

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-05-29 Thread liu ron
Hi, Jingsong Thanks for your review. We have tested it in TPC-DS case, and got a 12% gain overall when only supporting only Calc operator. In some queries, we even get more than 30% gain, it looks like an effective way. Best, Ron Jingsong Li 于2023年5月29日周一 14:33写道: > Thanks Ron for the

Re: [DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-05-29 Thread Jingsong Li
Thanks Ron for the proposal. Do you have some benchmark results for the performance improvement? I am more concerned about the improvement on Flink than the data in other papers. Best, Jingsong On Mon, May 29, 2023 at 2:16 PM liu ron wrote: > > Hi, dev > > I'd like to start a discussion about

[DISCUSS] FLIP-315: Support Operator Fusion Codegen for Flink SQL

2023-05-29 Thread liu ron
Hi, dev I'd like to start a discussion about FLIP-315: Support Operator Fusion Codegen for Flink SQL[1] As main memory grows, query performance is more and more determined by the raw CPU costs of query processing itself, this is due to the query processing techniques based on interpreted