ggershinsky
ggershinsky
yep, just the squash/merge button.
@shangxinli are you ok with this PR in its current form?
> > cc @ggershinsky @wgtmac let me know if you have concern to merge. > > Thanks for pinging me! I don't have any concern for now. Same here
Hi @shangxinli , I guess you are using a custom Parquet integration code. The way Iceberg calls the Parquet APIs today (in the master branch), doesn't allow for transparent activation...
Cool, I'll send them in a pr (after the parquet-mr-1.12 artifact is released; CI checks won't pass otherwise).
@steveloughran Thanks for this PR! When can we expect it to be merge-ready?
Per the [thread](https://lists.apache.org/thread/kttwbl5l7opz6nwb5bck2gghc2y3td0o), it'd be good to have this patch in 1.14.0 :) Otherwise, can take a very long time till the next one..
this PR is already covered there. But I mostly meant the thread title.
Per the discussion in https://issues.apache.org/jira/browse/PARQUET-2223, we need to converge on the design first. I left a number of comments in the design googledoc.