amogh-jahagirdar opened a new pull request, #6815: URL: https://github.com/apache/iceberg/pull/6815
Reverts to using stephenc findbugs dependency for Nullable based on @rdblue comment here: https://github.com/apache/iceberg/pull/6598/files#r1103888557 cc @danielcweeks since there is a differing view based off of https://github.com/apache/iceberg/pull/6598#discussion_r1096213570 . In the current PR, we're reverting back to using the stephenc dependency, because it appears the javax one from com.google.code.findbugs:jsr305 is banned. Let me know your thoughts on reverting this! To add a third option: It seems like the stephenc findbugs dependency has Nullable marked as deprecated, and based off https://stackoverflow.com/questions/19030954/cant-find-nullable-inside-javax-annotation spotbugs is the newer reccomended library to get the javax.annotation but this still just has a dependency on com.google.code.findbugs:jsr305. CC: @jackye1995 who is managing the release for 1.2.0. -- 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
