mgodave opened a new pull request #1515: Consolidate Pair implementations
URL: https://github.com/apache/incubator-pulsar/pull/1515
 
 
   ### Motivation
   
   While working in the codebase I noticed that there were at least three 
different implementations of a "Pair<A,B>" type being used. I decided to 
consolidate them into one implementation from commons-lang3. Please advise if 
it is somehow desirable to keep one of both of the deleted implementations.
   
   ### Modifications
   
   Replace:
   org.apache.bookkeeper.mledger.util.Pair
   org.apache.pulsar.common.util.collections.Pair
   
   With:
   org.apache.commons.lang3.tuple.Pair
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to