dramaticlly commented on PR #12844:
URL: https://github.com/apache/iceberg/pull/12844#issuecomment-2819025708

   > Interesting, is it all that you need to do Hive -> Iceberg conversion. 
Seems simple and make sense to me. cc @flyrain @dramaticlly for any thoughts
   
   Glad to hear RewriteTablePath can be used this way and I am ok to add a flag 
to control the behavior of saving file list location. I think currently it's 
tied to staging location and changing staging location will also influence 
where the metadata files will be saved on disk. 
   
   Also nit, I think your change include both Spark 3.4 and Spark 3.5, so you 
might want to reflect that in the PR title. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to