collado-mike commented on PR #321: URL: https://github.com/apache/polaris/pull/321#issuecomment-2380086779
> Yeah good point - I definitely also considered something along the lines of a "dry-run" parameter for this instead that could be generalized to other API requests. In this case when I started implementing it that way it became clear that there are enough deviations for the use case that it would be a bit misleading and messy to treat it as a dry run. A `HEAD` option would be cool - except that I don't think you're allowed to submit a request body. Maybe support a query parameter? The prospect of `VALIDATE` as a notification type doesn't really make sense to me. Ultimately, I think a collection of specific checks - privileges, table location, catalog type - will all be super useful. Though I suppose this notify API is apt to change anyway, so it probably doesn't hurt to add a new notification type now, even if it is a bit wonky -- 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]
