Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-05-06 Thread Ramin Gharib
Hi Flink Community, Following the discussion on FLIP-528: Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL, the proposal has been thoroughly discussed, and the main points have been addressed. So, I'll be starting the voting process tomorrow. Please review the final version of the FLIP

RE: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-05-01 Thread David Radley
at 08:42 To: dev@flink.apache.org Subject: [EXTERNAL] Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL Hi all, Thanks for the FLIP Ramin. I would be +1 for this. It creates a consistent experience for SQL users, because they only have to prefix their earlier SQL

Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-04-30 Thread Dawid Wysakowicz
t; >>> > >>> David Radley 于2025年4月28日周一 21:51写道: > >>> > >>>> Hi Ramin, > >>>> Thanks for making the changes. > >>>> As far as I can see the EXECUTE STATEMENT SET is Flink specific; so I > >> am > >>

Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-04-30 Thread Timo Walther
8 April 2025 at 13:41 To: dev@flink.apache.org Subject: [EXTERNAL] Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL Hello David, Many thanks for the response! I have moved the discussion to a new link. The FLIP number was wrong (now it is at 528). You can find the n

Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-04-30 Thread Gustavo de Morais
gt; > > not sure this is part of the SQL standard. So, I guess adding the > explain > > > syntax as an alias for the existing explain would be ok. > > > > > > I assume Flink decided not to use stored procedures syntax to run the > > > multiple in

RE: Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-04-29 Thread Gustavo de Morais
d. So, I guess adding the explain > > > syntax as an alias for the existing explain would be ok. > > > > > > I assume Flink decided not to use stored procedures syntax to run the > > > multiple inserts – which would be what I would have expected here. > > > >

Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-04-29 Thread Martijn Visser
ocedures syntax to run the > > multiple inserts – which would be what I would have expected here. > > > > Kind regards David. > > > > From: Ramin Gharib > > Date: Monday, 28 April 2025 at 13:41 > > To: dev@flink.apache.org > > Subject: [EXTERNAL] R

Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-04-28 Thread Ron Liu
vid. > > From: Ramin Gharib > Date: Monday, 28 April 2025 at 13:41 > To: dev@flink.apache.org > Subject: [EXTERNAL] Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE > STATEMENT SET Syntax in Flink SQL > Hello David, > > Many thanks for the response! I have moved the discu

RE: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-04-28 Thread David Radley
procedures syntax to run the multiple inserts – which would be what I would have expected here. Kind regards David. From: Ramin Gharib Date: Monday, 28 April 2025 at 13:41 To: dev@flink.apache.org Subject: [EXTERNAL] Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-04-28 Thread Ramin Gharib
Hello David, Many thanks for the response! I have moved the discussion to a new link. The FLIP number was wrong (now it is at 528). You can find the new link below https://lists.apache.org/thread/28llyjdks9hrj5rvpbxhd3zd7bfqwrw8 I have fixed the inconsistency in the text. Thanks for raising the

Re: [DISCUSS] FLIP-527 Support EXPLAIN EXECUTE STATEMENT SET Syntax in Flink SQL

2025-04-28 Thread David Radley
Hi Ramin, Thanks for the Flip. In the text you say EXECUTE STATEMENT SET BEGIN INSERT INTO `sales` SELECT ...; INSERT INTO `products` SELECT ...; END; Twice, I think you are intending the 2nd text to be different. Maybe EXPLAIN STATEMENT SET I am worried that in the SQL standard EXECUTE a