Hi Parth and Andy, Thank you both for the context. I will add this topic to the agenda for the next Iceberg community sync to discuss the pending work and determine the best way forward.
~ Anurag On Fri, Mar 20, 2026 at 3:18 PM Parth Chandra <[email protected]> wrote: > 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. >> >
