szehon-ho commented on PR #5688: URL: https://github.com/apache/iceberg/pull/5688#issuecomment-1234565058
Yea, I personally think it would be great to make this configurable, as maybe change the default 20 as I feel it is a bit much. What do you think to put it in TableProperties? Just going over CatalogProperties and TableProperties, I feel TableProperties has more similar configs like commit.retry.num-retries. How about in TableProperties : read.retries.metadata-refresh? On CatalogProperties there's 'table-default' and 'table-override' that will allow default/override for all flag to be set on Catalog as well, so it will accomplish what this pr tries to do as well. Another question is whether all catalog will support it. I think all can, but not sure RestCatalog, cc @rdblue if there's concerns adding such a flag in either Catalog/TableProperties for that. -- 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]
