I agree that method randomEntry() can be deprecated. However, in that case we should also deprecate RandomEvictionPolicy, which probably is used in some tests.
Does anyone see any potential issues here? D. On Thu, Oct 15, 2015 at 7:37 AM, Sergi Vladykin <[email protected]> wrote: > Agree, it is doubtfully useful. > > Sergi > > 2015-10-15 17:26 GMT+03:00 Roman <[email protected]>: > > > And another ticket for randomEntry() > > https://issues.apache.org/jira/browse/IGNITE-1180 > > > > > > On Thursday, October 15, 2015 9:47 PM, Vladimir Ozerov < > > [email protected]> wrote: > > > > > > Igniters, > > > > We have the ticket about broken randomEntry() method: > > https://issues.apache.org/jira/browse/IGNITE-512 > > > > This method can return null even if cache is not empty. And it is even > > impossible to return an entry in constant time provided that concurrent > > deletes and expirations could occur. > > > > Do we really need this method? As for me it is better to deprecate it and > > do not spend any efforts trying to make it work. > > > > Thoughts? > > > > > > > > >
