kbendick commented on issue #4828: URL: https://github.com/apache/iceberg/issues/4828#issuecomment-1138771607
In an effort to keep the OpenAPI spec slimmed down, and because this class isn't implemented, we've decided to not add it for the moment. The use case for `AssignUUID` is almost somewhat theoretical. If people have tables that are _very_ old (e.g. from the 0.9.x days or from when the project was incubating), they might not have UUIDs (as the spec initially didn't call for them). Because we don't think this will be a common use case within OSS (nor within any of my own work), for simplicities sake we've decided to leave it out. That's also why the logic is not fully implemented. If somebody eventually _does_ need this (e.g. for registering tables or for updating table UUIDs for some reason other than during a normal commit workflow), then we can also add it to the spec. As @singhpk234 pointed out, the class I was actually looking for was `RemoveSnapshotRef`. Maybe Prashant should just take over my job for me 🙈. -- 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]
