[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [VOTE] Release Apache Commons RNG 1.1 based on RC6


On Tue, 7 Aug 2018 15:11:30 -0600, Gary Gregory wrote:
Hi All:

Is this failure expected:

[INFO] --- clirr-maven-plugin:2.8:check (default-cli) @
commons-rng-sampling ---
[INFO] Comparing to version: 1.0
[ERROR] 5001:

Removed org.apache.commons.rng.sampling.distribution.SamplerBase from the
list of superclasses
[ERROR] 5001: org.apache.commons.rng.sampling.distribution.PoissonSampler: Removed org.apache.commons.rng.sampling.distribution.SamplerBase from the
list of superclasses



The first, yes.[1]

The second, I overlooked.  Sorry.
The "SamplerBase" class was an easy way to not repeat boiler-plate
code and to avoid the additional indirection of composition.
The latter would have been cleaner but the choice was made amid
strong pressure (and unkind words) that the refactoring should not
loose 1% (!) of performance wrt the Commons Math implementations.[2]

The issue is only for classes that
1. inherit from "PoissonSampler",
2. call the protected methods in "SamplerBase".

I don't see any appropriate use-case but the new implementation is
not a drop-in remplacement. :-/

A fix would be to reinstate the base class and call "super(null)"
(but an application that does attempt to use the class as described
above will generate a NPE).
Calling "super(rng)" would fix the compatibility (by still allowing
"incorrect" usage).


[1] https://issues.apache.org/jira/browse/RNG-46
[2] Since then, additional RNGs were implemented that are ~40% to
    ~120% faster (depending on the type of value generated) than
    what exists in CM.



To unsubscribe, e-mail: dev-unsubscribe@xxxxxxxxxxxxxxxxxx
For additional commands, e-mail: dev-help@xxxxxxxxxxxxxxxxxx