lhotari opened a new pull request #9693:
URL: https://github.com/apache/pulsar/pull/9693


   ### Motivation
   
   - it is a bad practice to reference shaded classes in other modules.
   - Guava's Supplier class is a legacy class in Guava:
     - ["Legacy version of 
java.util.function.Supplier."](https://github.com/google/guava/blob/3fbc6f2db3a2673b95e69115cb41c4ad9bb7fe83/guava/src/com/google/common/base/Supplier.java#L21)
   
   ### Modifications
   
   - Replace the usage of the shaded Guava Supplier with 
`java.util.function.Supplier`. 
     - This is also instructed [in the javadoc of Guava's 
Supplier](https://github.com/google/guava/blob/3fbc6f2db3a2673b95e69115cb41c4ad9bb7fe83/guava/src/com/google/common/base/Supplier.java#L27-L30):
 
       - "As this interface extends java.util.function.Supplier, an instance of 
this type can be used as a java.util.function.Supplier directly. To use a 
java.util.function.Supplier in a context where a 
com.google.common.base.Supplier is needed, use supplier::get."
   


----------------------------------------------------------------
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]


Reply via email to