+1 On Thu, Jan 23, 2025 at 4:49 AM Honah J. <hon...@apache.org> wrote:
> +1, thanks Russell! > > Best regards, > Honah > > On Wed, Jan 22, 2025 at 6:49 PM Renjie Liu <liurenjie2...@gmail.com> > wrote: > >> +1, thanks Russell for driving this! >> >> On Thu, Jan 23, 2025 at 8:12 AM Russell Spitzer < >> russell.spit...@gmail.com> wrote: >> >>> That was a bit of what we discussed at the sync this morning. Whether we >>> should have a generic enable feature update for one way features that we >>> don’t have the ability to disable. We couldn’t come up with more examples >>> of features we actually wanted to add. I think I’d we have at least two >>> more it would make sense. >>> >>> On Wed, Jan 22, 2025 at 6:04 PM Daniel Weeks <dwe...@apache.org> wrote: >>> >>>> Just a minor question added to the PR. We're adding an explicit >>>> 'enable' as an update type and I wonder if it would be better to generalize >>>> it so that we don't have separate updates to disable/enable (more forward >>>> thinking as this is the first case quite like this). >>>> >>>> -Dan >>>> >>>> >>>> >>>> On Wed, Jan 22, 2025 at 3:55 PM Amogh Jahagirdar <2am...@gmail.com> >>>> wrote: >>>> >>>>> +1 Thanks Russell >>>>> >>>>> On Wed, Jan 22, 2025 at 4:50 PM rdb...@gmail.com <rdb...@gmail.com> >>>>> wrote: >>>>> >>>>>> +1 >>>>>> >>>>>> On Wed, Jan 22, 2025 at 2: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 >>>>>>> >>>>>>
