+1 (non-binding). On Wed, Apr 30, 2025 at 3:49 PM Daniel Weeks <[email protected]> wrote:
> +1 (binding) > > On Wed, Apr 30, 2025 at 1:38 PM Amogh Jahagirdar <[email protected]> wrote: > >> +1 (binding) >> >> On Wed, Apr 30, 2025 at 1:29 PM Anurag Mantripragada >> <[email protected]> wrote: >> >>> +1 (non-binding) >>> >>> ~ Anurag Mantripragada >>> >>> >>> >>> >>> >>> >>> >>> On Apr 30, 2025, at 11:44 AM, Ryan Blue <[email protected]> wrote: >>> >>> +1 (binding) >>> >>> On Wed, Apr 30, 2025 at 10:34 AM Huang-Hsiang Cheng >>> <[email protected]> wrote: >>> >>>> +1 (non-binding) >>>> >>>> Huang-Hsiang >>>> >>>> On Apr 30, 2025, at 10:06 AM, Yufei Gu <[email protected]> wrote: >>>> >>>> +1 >>>> Yufei >>>> >>>> >>>> On Wed, Apr 30, 2025 at 10:04 AM Russell Spitzer < >>>> [email protected]> wrote: >>>> >>>>> +1 >>>>> >>>>> On Wed, Apr 30, 2025 at 11:36 AM Szehon Ho <[email protected]> >>>>> wrote: >>>>> >>>>>> +1 >>>>>> >>>>>> Thanks >>>>>> Szehon >>>>>> >>>>>> On Wed, Apr 30, 2025 at 4:10 AM Eduard Tudenhöfner < >>>>>> [email protected]> wrote: >>>>>> >>>>>>> +1 (binding) >>>>>>> >>>>>>> On Tue, Apr 29, 2025 at 9:29 PM Ryan Blue <[email protected]> wrote: >>>>>>> >>>>>>>> Hi everyone, >>>>>>>> >>>>>>>> I’d like to propose merging PR 12162 >>>>>>>> <https://github.com/apache/iceberg/pull/12162/files> into the >>>>>>>> table spec for v3. The changes are a minimal set of additions needed to >>>>>>>> support table encryption schemes, including the scheme that we’re >>>>>>>> working >>>>>>>> on for table encryption with client-managed keys that will be a >>>>>>>> separate >>>>>>>> doc. >>>>>>>> >>>>>>>> There are two main changes: >>>>>>>> >>>>>>>> - Add a table of encryption-keys that stores encrypted keys >>>>>>>> - Add a key-id to snapshot metadata that identifies the key >>>>>>>> used to encrypt the snapshot >>>>>>>> >>>>>>>> These are the only changes needed for v3, which we want to keep >>>>>>>> separate from the encryption scheme. The table format is responsible >>>>>>>> for >>>>>>>> storing the encrypted key material and the key used for a snapshot. >>>>>>>> These >>>>>>>> changes are compatible with REST catalogs as well as older catalogs >>>>>>>> like >>>>>>>> Hive that manage the metadata JSON location. >>>>>>>> >>>>>>>> Please take a look and vote in the next 72 hours. >>>>>>>> >>>>>>>> [ ] +1 Add these changes to the spec >>>>>>>> [ ] +0 >>>>>>>> [ ] -1 I have questions and/or concerns >>>>>>>> >>>>>>>> Thanks, >>>>>>>> >>>>>>>> Ryan >>>>>>>> >>>>>>> >>>> >>>
