This behavior should definitely be fixed - exception should be thrown on
client node. I will update ticket. William, thanks for pointing this out.

--Yakov

2015-10-15 20:33 GMT+03:00 William Kemper <[email protected]>:

> I never filed a ticket, but for what it's worth, I tried using this method
> in a test scenario just last week from a _client_ node and it never
> returned (i.e., blocked forever).
>
> On Thu, Oct 15, 2015 at 8:41 AM, Dmitriy Setrakyan <[email protected]>
> wrote:
>
> > 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?
> > > >
> > > >
> > > >
> > > >
> > >
> >
>

Reply via email to