Return-Path: X-Original-To: apmail-ignite-dev-archive@minotaur.apache.org Delivered-To: apmail-ignite-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id E03CF182FB for ; Thu, 15 Oct 2015 15:41:56 +0000 (UTC) Received: (qmail 51800 invoked by uid 500); 15 Oct 2015 15:41:56 -0000 Delivered-To: apmail-ignite-dev-archive@ignite.apache.org Received: (qmail 51757 invoked by uid 500); 15 Oct 2015 15:41:56 -0000 Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ignite.apache.org Delivered-To: mailing list dev@ignite.apache.org Received: (qmail 51746 invoked by uid 99); 15 Oct 2015 15:41:56 -0000 Received: from mail-relay.apache.org (HELO mail-relay.apache.org) (140.211.11.15) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 15 Oct 2015 15:41:56 +0000 Received: from mail-oi0-f45.google.com (mail-oi0-f45.google.com [209.85.218.45]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id 16A3D1A045C for ; Thu, 15 Oct 2015 15:41:55 +0000 (UTC) Received: by oies66 with SMTP id s66so8708780oie.1 for ; Thu, 15 Oct 2015 08:41:54 -0700 (PDT) X-Gm-Message-State: ALoCoQlSiErUsC2ez6y+8He6XjQNvMwGuqHDZeKxTBajSEPintlyKQqmaslzsHo5AUBL+dXCTVE6 X-Received: by 10.202.86.214 with SMTP id k205mr5802206oib.115.1444923714764; Thu, 15 Oct 2015 08:41:54 -0700 (PDT) MIME-Version: 1.0 Received: by 10.202.223.67 with HTTP; Thu, 15 Oct 2015 08:41:15 -0700 (PDT) In-Reply-To: References: <531879978.833701.1444919200482.JavaMail.yahoo@mail.yahoo.com> From: Dmitriy Setrakyan Date: Thu, 15 Oct 2015 08:41:15 -0700 Message-ID: Subject: Re: Do we need "IgniteCache.randomEntry()" method? To: dev@ignite.apache.org Cc: Roman Content-Type: multipart/alternative; boundary=001a113d841abf1f2c0522268425 --001a113d841abf1f2c0522268425 Content-Type: text/plain; charset=UTF-8 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 wrote: > Agree, it is doubtfully useful. > > Sergi > > 2015-10-15 17:26 GMT+03:00 Roman : > > > And another ticket for randomEntry() > > https://issues.apache.org/jira/browse/IGNITE-1180 > > > > > > On Thursday, October 15, 2015 9:47 PM, Vladimir Ozerov < > > vozerov@gridgain.com> 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? > > > > > > > > > --001a113d841abf1f2c0522268425--