Thanks all!

>Would it be possible to include
https://github.com/apache/iceberg/pull/14153 to enable variant shredding

Regarding this one, I don't think it should be in a patch release since
it's exposing an API on the data file writer and not really a fix. Besides
it is technically possible already to just use the lower level parquet
writer with the variant shredding spec and then produce a data file from
that (it's a bit more clunky for sure which is why that API change in the
PR is helpful) but don't think it quite meets the criteria for being in a
minor release. That criteria is primarily based on fixes.

Thanks,
Amogh Jahagirdar


On Tue, Sep 23, 2025 at 10:13 AM Jean-Baptiste Onofré <j...@nanthrax.net>
wrote:

> Hi Amogh
>
> Thanks for starting the discussion!
>
> I will create the PR to "relax" the LICENSE/NOTICE (to avoid the
> update pain before release).
>
> Regards
> JB
>
> On Mon, Sep 22, 2025 at 6:29 PM Amogh Jahagirdar <2am...@gmail.com> wrote:
> >
> > Hey folks,
> >
> > Iceberg 1.10 was released 2 weeks ago and there was one issue around
> incorrect variant filtering reported that I think meets the criteria for a
> patch release. The fix PR is in (thank you Drew). I wanted to kick this
> discussion thread off in case folks had other issues in the 1.10 release
> that they think warrant a patch release.
> >
> > I also think this PR  is a good candidate for a patch release; this is
> for addressing a long-standing issue where closing the S3FileIO during an
> event like moving broadcast variables from memory to disk leads to an
> unexpected closing of the http client. There's still some discussion on the
> approach of the fix but there's general recognition that it's a legitimate
> issue, so I think it'd be ideal to get this in for a patch release as well.
> >
> > I've also created a milestone here.
> >
> > Thanks,
> >
> > Amogh Jahagirdar
>

Reply via email to