suyashpatel98 opened a new pull request, #12714:
URL: https://github.com/apache/pinot/pull/12714

   Fixes #12693. This PR upgrades commons-lang to commons-lang3 and 
commons-collections to commons-collections4. A few things to note:
   
   1. I did not upgrade commons-math to commons-math3 because PinotFourthMoment 
class extends FourthMoment from commons-math. In the newer version 
(commons-math3) FourthMoment has been made package-private and therefore, Pinot 
code cannot extend it. See 
[issue](https://issues.apache.org/jira/browse/MATH-694). A new issue will need 
to be created for this so that someone who has more context can look into it.
   2. I had to make some code changes because the new versions did introduce 
some breaking changes
   - org.apache.commons.lang.math.IntRange is being used by 
DataGenerationHelpers.java which has been deprecated in commons-lang3. Replaced 
it with org.apache.commons.lang3.Range
   - org.apache.commons.lang.math.RandomUtils has been moved to 
org.apache.commons.lang3.RandomUtils.


-- 
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: commits-unsubscr...@pinot.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to