Hi, Sergey. Thanks for the FLIP to improve the materialized table ddl. The proposal looks good to me overall.
Just one minor comment, about the Catalog#listMaterializedTables method signature, it should be a default method. Best, Ron Gustavo de Morais <gustavopg...@gmail.com> 于2025年8月19日周二 16:18写道： > Hey Sergey, > > The FLIP looks good. +1 as well. Thanks for driving it. > > Gustavo > > On Mon, 18 Aug 2025 at 23:08, Sergey Nuyanzin <snuyan...@gmail.com> wrote: > > > Hi Hao > > > > sure, added > > > > On Mon, Aug 18, 2025 at 10:48 PM Hao Li <h...@confluent.io.invalid> > wrote: > > > > > > Hi Sergey, > > > > > > Thanks for the FLIP. Can you add the modify distribution sql syntax to > > the > > > FLIP as well? Looks like you have `ModifyDistribution` but it's not in > > sql > > > syntax. > > > > > > Thanks, > > > Hao > > > > > > On Mon, Aug 18, 2025 at 6:23 AM Ramin Gharib <ramingha...@gmail.com> > > wrote: > > > > > > > Hi Sergey, > > > > > > > > Thanks for driving this! +1 from my side! > > > > > > > > Cheers, > > > > > > > > Ramin > > > > > > > > On Fri, Aug 15, 2025 at 3:42 PM Sergey Nuyanzin <snuyan...@gmail.com > > > > > > wrote: > > > > > > > > > Hi everyone, > > > > > I'd like to start a discussion of Support DISTRIBUTED BY/INTO for > > > > > Materialized tables [1]. > > > > > > > > > > In FLIP-376 there was introduced DISTRIBUTED BY/INTO functionality > to > > > > > support bucketing concept in Flink, then in FLIP-435 there were > > > > > introduced Materialized tables. However so far there is no support > > for > > > > > DISTRIBUTED BY/INTO for Materialized tables. This FLIP is going to > > > > > address this issue > > > > > > > > > > [1] > > > > > > > > > > > > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=378473269 > > > > > [2] > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/FLINK/FLIP-376%3A+Add+DISTRIBUTED+BY+clause > > > > > [3] > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/FLINK/FLIP-435%3A+Introduce+a+New+Materialized+Table+for+Simplifying+Data+Pipelines > > > > > > > > > > > > > > > > > > > > -- > > > > > Best regards, > > > > > Sergey > > > > > > > > > > > > > > > > > -- > > Best regards, > > Sergey > > >
