Looks like we are unanimous now, I can wait another day just in case anyone has last minute comments and I"ll merge tomorrow.
On Thu, Jan 23, 2025 at 3:55 PM Daniel Weeks <dwe...@apache.org> wrote: > +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 >>>>>> >>>>>