virajjasani edited a comment on pull request #3503: URL: https://github.com/apache/hadoop/pull/3503#issuecomment-931355295
@steveloughran I think backport might be bit painful for this one specifically given that VisibleForTesting is being used at so many places. I was thinking to restrict this to 3.4.0 only. Are you fine with it? Even previous Guava replacement work for `Sets` and `Lists` are restricted to trunk branch only so far because both of them are also being used at several places. Moreover, these changes are not bugfixes, and the actual removal of guava dependency (if possible) can happen with next major release (4.0.0) only, so we are good for now to keep these changes in trunk only if you are fine with it. -- 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]
