+1(binding). LGTM, esp. given that the standard positional deletes with (filename, row-position) will continue to exist.
Yufei On Wed, Sep 10, 2025 at 5:37 PM Yuya Ebihara <[email protected]> wrote: > +1 (non-binding) > > On Thu, Sep 11, 2025 at 6:06 AM Amogh Jahagirdar <[email protected]> wrote: > >> +1, thanks Peter >> >> >> >> On Wed, Sep 10, 2025 at 2:08 PM Wing Yew Poon <[email protected]> >> wrote: >> >>> +1 (non-binding) >>> >>> On Wed, Sep 10, 2025 at 12:48 PM Ryan Blue <[email protected]> wrote: >>> >>>> +1 to removing this from the Java implementation and API. >>>> >>>> On Wed, Sep 10, 2025 at 10:08 AM Steven Wu <[email protected]> >>>> wrote: >>>> >>>>> +1 >>>>> >>>>> On Wed, Sep 10, 2025 at 9:30 AM Russell Spitzer < >>>>> [email protected]> wrote: >>>>> >>>>>> +1 >>>>>> >>>>>> On Wed, Sep 10, 2025 at 10:58 AM Péter Váry < >>>>>> [email protected]> wrote: >>>>>> >>>>>>> The Position Deletes With Row (PDWR) feature, originally introduced >>>>>>> in the Iceberg V2 specification, has been deprecated in V3. >>>>>>> >>>>>>> Following the discussion in the previous thread [1], we propose >>>>>>> deprecating PDWR in the Java implementation starting with the Iceberg >>>>>>> 1.11.0 release. We plan to remove PDWR support entirely from the Java >>>>>>> implementation beginning with Iceberg 1.12.0. >>>>>>> >>>>>>> While PDWR will remain part of the V2 spec, the Java implementation >>>>>>> will no longer write PDWR delete files. It will only support reading the >>>>>>> position and file name columns from previously created delete files or >>>>>>> those generated by other engines. >>>>>>> >>>>>>> For reference, the proposed changes are available here [2]. >>>>>>> >>>>>>> Please cast your vote. >>>>>>> >>>>>>> Thanks, >>>>>>> Peter >>>>>>> >>>>>>> [1] - >>>>>>> https://lists.apache.org/thread/8jw6pb2vq3ghmdqf1yvy8n5n6gg1fq5s >>>>>>> [2] - https://github.com/apache/iceberg/pull/14045 >>>>>>> >>>>>>
