[GitHub] [iceberg] ajantha-bhat commented on pull request #6789: Nessie: Add refresh for catalog APIs

["ajantha-bhat (via GitHub)" <gi...@apache.org>]

ajantha-bhat commented on PR #6789:
URL: https://github.com/apache/iceberg/pull/6789#issuecomment-1434697865

   > Sure this does not break the expected-hash calculations for commits?
   
   I just added a testcase for commit from multiple catalogs on same branch + same table. It passes. 
   So, I think there is no impact. 
   Also, expected hash is passed after the refresh operation from NessieCatalog. So, the client won't be modified after passing expected hash in the same flow.
   
   @snazy: Do you see any problem with the current changes? Do you feel we should only modify `listTables, listNamespaces, listNamespaceMetadata, removeProperties,  setProperties` instead of refreshing all the APIs? 
   


-- 
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