RussellSpitzer commented on PR #10766: URL: https://github.com/apache/iceberg/pull/10766#issuecomment-2253554397
My only worry here is that RevAPI is missing this as a public api method change because it's in Scala but we are changing the named exception being thrown and thus changing a method signature. At the same time I think this is pretty much private code, so not a big deal. Any way if we could put this off till 4.0 I would still do that. But no strong feelings here -- 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
