shanielh commented on PR #11887: URL: https://github.com/apache/iceberg/pull/11887#issuecomment-2568980615
> I think generally we wouldn't want to introduce new API concepts unless there is some usage of that API within the core library itself (Otherwise we are basically just opening up technical debt without tests or benefit). Do we have plans for introducing a usage in the repo somewhere? > > > > At minimum I think we would want tests utilizing this API but I think a practical application would also be required as a justification. I don't think there's a utilisation to the API in the core, my practical use is to check whether all files are already deleted or not. This information already lies in the ValidationException, but it requires me to do string manipulation in order to get the information. As a general rule of thumb, I suggest all information that lies in the exception message should be somehow accessible via API. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
