kbendick commented on PR #4681:
URL: https://github.com/apache/iceberg/pull/4681#issuecomment-1115529855
> > I'm not sure we want to force all future table operations to include a
literal location (or have to return one). So we may want to have the logging in
the operations (HiveTableOperations, HadoopTableOperations, ... ) themselves?
>
> Good point. But I think a table version string will be there even for a
future catalog without a metatdata.json file. It can easily support that, with
minor word change. like `Successfully committed to table {} with the new
metadata location({})` -> `Successfully committed to table {} with the new
version ({})`
>
> Hi @kbendick, what do you think from the perspective of rest API catalog?
For the current REST catalog, a file should be loggable. That said, I do
agree that it's probably best not to force catalogs to do so.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
