+1 Sorry to have missed the discussion, but I'm onboard with the proposed changes.
On Thu, Jan 23, 2025 at 11:27 AM Yufei Gu <flyrain...@gmail.com> wrote: > +1 > Yufei > > > On Thu, Jan 23, 2025 at 11:05 AM huaxin gao <huaxin.ga...@gmail.com> > wrote: > >> +1 (non binding) >> >> Thanks Russell. >> >> On Thu, Jan 23, 2025 at 10:55 AM Fokko Driesprong <fo...@apache.org> >> wrote: >> >>> +1 >>> >>> Thanks Russell >>> >>> Op do 23 jan 2025 om 18:47 schreef Aihua Xu <aihu...@gmail.com>: >>> >>>> + (non binding). >>>> >>>> Thanks Russell. >>>> >>>> On Thu, Jan 23, 2025 at 2:05 AM Jean-Baptiste Onofré <j...@nanthrax.net> >>>> wrote: >>>> >>>>> +1 (non binding) >>>>> >>>>> Regards >>>>> JB >>>>> >>>>> On Wed, Jan 22, 2025 at 11:51 PM Russell Spitzer >>>>> <russell.spit...@gmail.com> wrote: >>>>> > >>>>> > Hey Y'all >>>>> > >>>>> > Yet another Row Lineage Spec update. This adds a MetadataUpdate >>>>> EnableRowLineage to the REST Spec. We briefly talked today >>>>> > about an alternative EnableFeature(Feature Name) API instead but in >>>>> the absence of other features it doesn't seem >>>>> > like that's really a requirement now. >>>>> > >>>>> > I agreed that if we ever do have another feature we want to enable >>>>> in a similar way I would take the blame for adding >>>>> > this API rather than a generic one. >>>>> > >>>>> > That said please take a look >>>>> > >>>>> > https://github.com/apache/iceberg/pull/12050 >>>>> > >>>>> > >>>>> > Note: We only allow enabling row lineage, it cannot be disabled. >>>>> > >>>>> > Thanks for your time, >>>>> > Russ >>>>> >>>>