Ryan Blue

Did you mean to use the new method somewhere? Also, I don't think that we want to use "unsafe" in the name. This is not unsafe, it is a different...

This looks like the right fix to me. Thanks, @singhpk234! Could you also add tests for the other FileIO implementations?

Thanks, @singhpk234!

I think it is good to note that the default is the data value, but document the settings exist.

> Not sure if there's an easier way to do it, but essentially our `RESTClient` requires the body to be a `RESTRequest` and we can't make `ScanReport` (part of `iceberg-api`)...

Nice work, @nastra! This looks ready so I merged it.

@Fokko, sorry, but I think I was wrong in my initial review. I think instead of renaming these, we actually want to rename the fields that we write so that...

I'm interested to hear what @szehon-ho and @RussellSpitzer think about this. My initial reaction is that this is not something that we should change. We don't want to disable AQE...

@manuzhang, it seems reasonable to create a session for the entire rewrite, not just each Spark submission. Is that what was happening before?

> Do you mean session was created for the entire rewrite before? I'm asking you what the behavior was before this change that you want to fix.