Re: [PR] Core: Skip unnecessary metadata refresh after merge append [iceberg]

["gaborkaszab (via GitHub)" <gi...@apache.org>]

gaborkaszab commented on PR #14709:
URL: https://github.com/apache/iceberg/pull/14709#issuecomment-3595503224

   Hey @flyrain @amogh-jahagirdar @nastra ,
   You might be interested in this change (I found you active within this area) where I found that when I do `table.newAppend().appendFile(...).commit()` then there is an excess load table request going to the REST catalog. In case I don't miss anything, I think we can skip this step because RESTTableOperations.commit() already refreshed table metadata.
   
   Note, I also see an unconditional table refresh in `CherryPickOperation.updateEvent()` but I didn't have the time to investigate that one further. In case there is support for the current enhancement, I can take a look at the cherry pick one in a separate PR.


-- 
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-unsubscribe@iceberg.apache.org

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


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