cnauroth commented on PR #7904: URL: https://github.com/apache/hadoop/pull/7904#issuecomment-3234168347
@slfan1989 , thanks for the review! The reason for the last Yetus failure was that I didn't have the right match for the Guava version number used by the GCS SDK. I pushed up an update. I also added more comments to explain what's going on for future maintainers. I'll wait for a clean Yetus run before committing. Regarding the old Guava version, I honestly don't remember why it's there, considering we now have hadoop-thirdparty. Maybe it was an assumption that old client projects had come to rely on it as a transitive dependency? There has been a policy that exposed dependencies are treated as public/stable: https://hadoop.apache.org/docs/current/hadoop-project-dist/hadoop-common/Compatibility.html#Java_Classpath Maybe 3.5.0 is an acceptable version boundary to remove this. I've been planning to start a separate discussion. -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
