David Phillips

Thanks for explaining. We should change `LocalFileIterator` to ignore those exceptions by not returning the file entry. Listing files is not atomic, so this is an expected race condition that...

I'm not sure why the original code used the async version of the interface, but we can use the synchronous [GlueClient](https://sdk.amazonaws.com/java/api/latest/software/amazon/awssdk/services/glue/GlueClient.html) in the new version.

That makes sense. Let's change Iceberg to use synchronous, but use async for Hive. I note that the V1 client actually used an executor internally, but with V2 we can...

If there is agreement, I can prototype the simple version.

I started hacking on this and it turns out to be rather complicated since it doesn't fit the existing model at all.

This is the API and test I started with: https://gist.github.com/electrum/5b08813442be8b2d348293ce0ddd2423 The extension model is based on a method annotation mapping to a `Handler`. This is different because it needs to...

Any update on this? We'd like to include this with [Presto](https://prestosql.io/) and having it in Maven Central would make that easier. Thanks

@hboutemy It is merged now and will be in the 449 release

We could change this to write to a temporary file and then rename. We could name the temporary file `.tmp.$RANDOM.$ORIGINAL`

I put up a fix in #21815 For inline functions and function creation, we directly analyze the original AST. The formatting round-trip was an oversight due to using the same...