kbendick commented on pull request #3770: URL: https://github.com/apache/iceberg/pull/3770#issuecomment-1004465913
> @jackye1995, it looks like we're mostly in agreement now: > > > 1. if it's better to have specific operation routes for non-RESTful operations > > 2. head table/namespace operation with response, if we do that what set of fields should be in the response > > 3. if pagination of response is needed when there are too many metadata log or snapshots > > 4. progress tracking for table purge > > 5. drop namespace cascade > > For 1, I think we're just trying to decide on the path for the `/tables/rename` route, right? I like the simplicity of `tables` and I don't think it will be confusing to have operation routes there. That's where I'd add multi-table transactions as well, which don't cleanly fit into a single REST resource. > > For 2, I think we should remove the HEAD request for namespaces and leave the tables HEAD request without response bodies. We're probably agreed? > > 3, 4, and 5 are not strictly required so I like the idea of adding them in follow-ups in parallel. That way we can start iterating on other areas with this as the basis. I'm in agreement on 1, that `/tables/rename` makes sense to me. It's just referencing tables in general. I've gone ahead and removed the HEAD request for namespaces. We can always add it back in in the future if need be. I agree that returning the whole table object for an existence check might be relatively expensive, and so it makes sense to keep the HEAD check for tables. I'm still not opposed to adding response bodies to the HEAD requests eventually, but that can be a topic for another day if need be. And then agreed on 3,4,5. The sooner we can get the initial document in, the sooner we can make smaller changes that are more digestible and can also work more in parallel 😄 -- 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]
