You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/12/16 14:24:33 UTC

[GitHub] [accumulo] DomGarguilo commented on pull request #3129: Replace copied HostAndPort with guava

DomGarguilo commented on PR #3129:
URL: https://github.com/apache/accumulo/pull/3129#issuecomment-1354932330

   > I saw you removed the HostAndPort test but I would think we should keep it and update it because one of the main things that was tested was the comparator functionality which doesn't exist in the Guava version so that still needs to be tested.
   
   Good point. Maybe the test should be renamed to HostAndPortComparatorTest or something like that.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org