The PR was closed because it required the Comet code that has now been removed. Even with that code reinstated, the PR is out of date and will likely need to be updated to make it work with the latest code base. There is also a PR for deleted rows support [1] which will also need to be updated.
[1] https://github.com/apache/iceberg/pull/14062 On Fri, Mar 20, 2026 at 10:50 AM Andy Grove <[email protected]> wrote: > Hi Anurag, > > I have no objection to my PR being reverted to allow for more > discussion. I think that would make sense. > > My only concern is that I am not sure how we would "fix the broken > code". My understanding is that there was a PR that aimed to do that > [1] but it was just closed after 7 months and 98 comments. Parth can > likely add more context here though. > > I'm open to suggestions on the best way to handle this. > > Thanks, > > Andy. > > > [1] https://github.com/apache/iceberg/pull/13786 > > > On Fri, Mar 20, 2026 at 11:28 AM Anurag Mantripragada > <[email protected]> wrote: > > > > Thanks for the PR, Andy. > > > > I see that it has been merged, but I feel we haven't yet had the > opportunity to discuss this in detail within the Iceberg community. Kevin > and I were discussing this offline and feel that removing the broken code > in its current state makes it difficult to restore in the future. > > > > I propose the following: > > - Revert the removal PR. > > - Fix the broken code and include it in a patch release so we freeze a > working version. > > - Discuss the removal during the next Iceberg community sync on April 1. > > - If we reach a consensus, re-apply the removal PR. > > > > What do you think? > > > > ~ Anurag > > > > On Wed, Mar 18, 2026 at 9:54 AM Andy Grove <[email protected]> > wrote: > >> > >> I have a draft PR up to remove the current Comet integration. I am > waiting for CI to finish before marking it as ready for review. > >> > >> https://github.com/apache/iceberg/pull/15674 > >> > >> On Wed, Mar 18, 2026 at 10:43 AM Kevin Liu <[email protected]> > wrote: > >>> > >>> Thanks for the update, Andy! > >>> It looks like most of the Comet code lives in the Spark integration > [1]. We're currently working on the next 1.11 release. I know the removal > isn't time-sensitive, but just wanted to mention it. > >>> > >>> Best, > >>> Kevin Liu > >>> > >>> [1] > https://grep.app/search?f.repo=apache%2Ficeberg&f.repo.pattern=apache%2Ficeberg&q=comet > >>> > >>> On Wed, Mar 18, 2026 at 8:24 AM Andy Grove <[email protected]> > wrote: > >>>> > >>>> Hi, > >>>> > >>>> The Comet community has made good progress with the iceberg-rust > integration and we now believe that this is the best path forward for > Comet, so are no longer going to be working on the integration with > Iceberg-Java. I posted a comment on the discussion issue as well [1]. > >>>> > >>>> Thanks for the community support with all of this. We will follow up > with a PR soon to remove the current (broken) Comet integration from > Iceberg-Java. > >>>> > >>>> Thanks, > >>>> > >>>> Andy. >
