Re: Re: [DISCUSS] Remove legacy Paimon (TableStore) doc link from Flink web navigation
Hi Marton and Martijn, I have removed the link to the legacy Paimon (flink-table-store) but only left a link to the incubating-paimon doc. Please move to the PR review[1] for quick discussions. [1] https://github.com/apache/flink-web/pull/665 Best Yun Tang From: Márton Balassi Sent: Monday, October 23, 2023 17:06 To: dev@flink.apache.org ; myas...@live.com Cc: d...@paimon.apache.org Subject: Re: Re: [DISCUSS] Remove legacy Paimon (TableStore) doc link from Flink web navigation Hi all, Thanks for your responses. @Jingsong Li: Thanks for the reference to the web PR, I missed that. @Yun Tang: Thanks, I prefer simply removing the TableStore link from the documentation navigation of Flink, as it is not a subproject of Flink anymore - it is now its own project. It has had 2 of its own releases over a ~half a year period. I am all for having a proper links to Paimon, for example we could create a "Sister Projects" subsection in the About section of the flink.apache.org<http://flink.apache.org> webpage and have a paragraph of intro/links there or simply add Paimon related content to the Table connector docs [1]. We can make these 2 changes separately, but ideally they should be merged relatively close in time. Would you be open to updating your original PR [2] to simply remove the links or would you like me to do it instead? I am happy to review your change if you have a proposal of where to include Paimon. [1] https://nightlies.apache.org/flink/flink-docs-release-1.17/docs/connectors/table/overview/ [2] https://github.com/apache/flink-web/pull/665 On Sat, Oct 21, 2023 at 7:33 AM Yun Tang mailto:myas...@live.com>> wrote: Hi devs, I am supporting to update the links. As the PR [1] author, I originally wanted to keep the legacy link to the Flink table store (0.3) as that was part of the history of Flink. And I plan to add the new Paimon's main branch link to tell users from Flink that this is the latest version. WDYT? [1] https://github.com/apache/flink-web/pull/665 Best Yun Tang From: junjie201...@gmail.com<mailto:junjie201...@gmail.com> mailto:junjie201...@gmail.com>> Sent: Friday, October 20, 2023 12:20 To: Jing Ge ; dev mailto:dev@flink.apache.org>> Cc: dev mailto:d...@paimon.apache.org>> Subject: Re: Re: [DISCUSS] Remove legacy Paimon (TableStore) doc link from Flink web navigation +1 On Tue, Oct 17, 2023 at 11:13 AM Yong Fang mailto:zjur...@gmail.com>> wrote: > +1 > > On Tue, Oct 17, 2023 at 4:52 PM Leonard Xu > mailto:xbjt...@gmail.com>> wrote: > > > +1 > > > > > > > 2023年10月17日 下午4:50，Martijn Visser > > > mailto:martijnvis...@apache.org>> 写道： > > > > > > +1 > > > > > > On Tue, Oct 17, 2023 at 10:34 AM Jingsong Li > > > mailto:jingsongl...@gmail.com>> > > wrote: > > >> > > >> Hi marton, > > >> > > >> Thanks for driving. +1 > > >> > > >> There is a PR to remove legacy Paimon > > >> https://github.com/apache/flink-web/pull/665 , but it hasn't been > > >> updated for a long time. > > >> > > >> Best, > > >> Jingsong > > >> > > >> On Tue, Oct 17, 2023 at 4:28 PM Márton Balassi > > >> mailto:mbala...@apache.org>> > > wrote: > > >>> > > >>> Hi Flink & Paimon devs, > > >>> > > >>> The Flink webpage documentation navigation section still lists the > > outdated TableStore 0.3 and master docs as subproject docs (see > > attachment). I am all for advertising Paimon as a sister project of > Flink, > > but the current state is misleading in multiple ways. > > >>> > > >>> I would like to remove these obsolete links if the communities agree. > > >>> > > >>> Best, > > >>> Marton > > > > >
Re: Re: [DISCUSS] Remove legacy Paimon (TableStore) doc link from Flink web navigation
Hi all, Thanks for your responses. @Jingsong Li: Thanks for the reference to the web PR, I missed that. @Yun Tang: Thanks, I prefer simply removing the TableStore link from the documentation navigation of Flink, as it is not a subproject of Flink anymore - it is now its own project. It has had 2 of its own releases over a ~half a year period. I am all for having a proper links to Paimon, for example we could create a "Sister Projects" subsection in the About section of the flink.apache.org webpage and have a paragraph of intro/links there or simply add Paimon related content to the Table connector docs [1]. We can make these 2 changes separately, but ideally they should be merged relatively close in time. Would you be open to updating your original PR [2] to simply remove the links or would you like me to do it instead? I am happy to review your change if you have a proposal of where to include Paimon. [1] https://nightlies.apache.org/flink/flink-docs-release-1.17/docs/connectors/table/overview/ [2] https://github.com/apache/flink-web/pull/665 On Sat, Oct 21, 2023 at 7:33 AM Yun Tang wrote: > Hi devs, > > I am supporting to update the links. As the PR [1] author, I originally > wanted to keep the legacy link to the Flink table store (0.3) as that was > part of the history of Flink. And I plan to add the new Paimon's main > branch link to tell users from Flink that this is the latest version. WDYT? > > [1] https://github.com/apache/flink-web/pull/665 > > Best > Yun Tang > > > From: junjie201...@gmail.com > Sent: Friday, October 20, 2023 12:20 > To: Jing Ge ; dev > Cc: dev > Subject: Re: Re: [DISCUSS] Remove legacy Paimon (TableStore) doc link from > Flink web navigation > > +1 > > On Tue, Oct 17, 2023 at 11:13 AM Yong Fang wrote: > > > +1 > > > > On Tue, Oct 17, 2023 at 4:52 PM Leonard Xu wrote: > > > > > +1 > > > > > > > > > > 2023年10月17日 下午4:50，Martijn Visser 写道： > > > > > > > > +1 > > > > > > > > On Tue, Oct 17, 2023 at 10:34 AM Jingsong Li > > > > wrote: > > > >> > > > >> Hi marton, > > > >> > > > >> Thanks for driving. +1 > > > >> > > > >> There is a PR to remove legacy Paimon > > > >> https://github.com/apache/flink-web/pull/665 , but it hasn't been > > > >> updated for a long time. > > > >> > > > >> Best, > > > >> Jingsong > > > >> > > > >> On Tue, Oct 17, 2023 at 4:28 PM Márton Balassi > > > > wrote: > > > >>> > > > >>> Hi Flink & Paimon devs, > > > >>> > > > >>> The Flink webpage documentation navigation section still lists the > > > outdated TableStore 0.3 and master docs as subproject docs (see > > > attachment). I am all for advertising Paimon as a sister project of > > Flink, > > > but the current state is misleading in multiple ways. > > > >>> > > > >>> I would like to remove these obsolete links if the communities > agree. > > > >>> > > > >>> Best, > > > >>> Marton > > > > > > > > >
Re: Re: [DISCUSS] Remove legacy Paimon (TableStore) doc link from Flink web navigation
Hi Yun, I think this discussion thread was about removing the links, I don't think we should keep them. For history purposes, there's always Git where things can be retrieved from. Best regards, Martijn On Sat, Oct 21, 2023 at 7:33 AM Yun Tang wrote: > > Hi devs, > > I am supporting to update the links. As the PR [1] author, I originally > wanted to keep the legacy link to the Flink table store (0.3) as that was > part of the history of Flink. And I plan to add the new Paimon's main branch > link to tell users from Flink that this is the latest version. WDYT? > > [1] https://github.com/apache/flink-web/pull/665 > > Best > Yun Tang > > > From: junjie201...@gmail.com > Sent: Friday, October 20, 2023 12:20 > To: Jing Ge ; dev > Cc: dev > Subject: Re: Re: [DISCUSS] Remove legacy Paimon (TableStore) doc link from > Flink web navigation > > +1 > > On Tue, Oct 17, 2023 at 11:13 AM Yong Fang wrote: > > > +1 > > > > On Tue, Oct 17, 2023 at 4:52 PM Leonard Xu wrote: > > > > > +1 > > > > > > > > > > 2023年10月17日 下午4:50，Martijn Visser 写道： > > > > > > > > +1 > > > > > > > > On Tue, Oct 17, 2023 at 10:34 AM Jingsong Li > > > wrote: > > > >> > > > >> Hi marton, > > > >> > > > >> Thanks for driving. +1 > > > >> > > > >> There is a PR to remove legacy Paimon > > > >> https://github.com/apache/flink-web/pull/665 , but it hasn't been > > > >> updated for a long time. > > > >> > > > >> Best, > > > >> Jingsong > > > >> > > > >> On Tue, Oct 17, 2023 at 4:28 PM Márton Balassi > > > wrote: > > > >>> > > > >>> Hi Flink & Paimon devs, > > > >>> > > > >>> The Flink webpage documentation navigation section still lists the > > > outdated TableStore 0.3 and master docs as subproject docs (see > > > attachment). I am all for advertising Paimon as a sister project of > > Flink, > > > but the current state is misleading in multiple ways. > > > >>> > > > >>> I would like to remove these obsolete links if the communities agree. > > > >>> > > > >>> Best, > > > >>> Marton > > > > > > > >
Re: Re: [DISCUSS] Remove legacy Paimon (TableStore) doc link from Flink web navigation
Hi devs, I am supporting to update the links. As the PR [1] author, I originally wanted to keep the legacy link to the Flink table store (0.3) as that was part of the history of Flink. And I plan to add the new Paimon's main branch link to tell users from Flink that this is the latest version. WDYT? [1] https://github.com/apache/flink-web/pull/665 Best Yun Tang From: junjie201...@gmail.com Sent: Friday, October 20, 2023 12:20 To: Jing Ge ; dev Cc: dev Subject: Re: Re: [DISCUSS] Remove legacy Paimon (TableStore) doc link from Flink web navigation +1 On Tue, Oct 17, 2023 at 11:13 AM Yong Fang wrote: > +1 > > On Tue, Oct 17, 2023 at 4:52 PM Leonard Xu wrote: > > > +1 > > > > > > > 2023年10月17日 下午4:50，Martijn Visser 写道： > > > > > > +1 > > > > > > On Tue, Oct 17, 2023 at 10:34 AM Jingsong Li > > wrote: > > >> > > >> Hi marton, > > >> > > >> Thanks for driving. +1 > > >> > > >> There is a PR to remove legacy Paimon > > >> https://github.com/apache/flink-web/pull/665 , but it hasn't been > > >> updated for a long time. > > >> > > >> Best, > > >> Jingsong > > >> > > >> On Tue, Oct 17, 2023 at 4:28 PM Márton Balassi > > wrote: > > >>> > > >>> Hi Flink & Paimon devs, > > >>> > > >>> The Flink webpage documentation navigation section still lists the > > outdated TableStore 0.3 and master docs as subproject docs (see > > attachment). I am all for advertising Paimon as a sister project of > Flink, > > but the current state is misleading in multiple ways. > > >>> > > >>> I would like to remove these obsolete links if the communities agree. > > >>> > > >>> Best, > > >>> Marton > > > > >
