virajjasani commented on pull request #2985:
URL: https://github.com/apache/hadoop/pull/2985#issuecomment-837060377
> However, since you already got an idea of what wrappers need to be
implemented to replace the {{Guava.sets}}, it may be better to create a PR only
for the wrappers (including TreeSets), then once that code is merged, you can
replace the guava.sets in every module.
I just saw this comment when page refreshed. I think this idea also looks
nice and clean, but I believe if we cover hadoop-common and hadoop-tools with
initial change, that one also looks clean and we have few usage in place for
first commit. WDYT?
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
