Return-Path: X-Original-To: apmail-lucene-dev-archive@www.apache.org Delivered-To: apmail-lucene-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 1C3A1DA39 for ; Mon, 10 Sep 2012 14:36:51 +0000 (UTC) Received: (qmail 47119 invoked by uid 500); 10 Sep 2012 14:36:49 -0000 Delivered-To: apmail-lucene-dev-archive@lucene.apache.org Received: (qmail 47067 invoked by uid 500); 10 Sep 2012 14:36:49 -0000 Mailing-List: contact dev-help@lucene.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@lucene.apache.org Delivered-To: mailing list dev@lucene.apache.org Received: (qmail 47060 invoked by uid 99); 10 Sep 2012 14:36:49 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 10 Sep 2012 14:36:49 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of rcmuir@gmail.com designates 209.85.220.176 as permitted sender) Received: from [209.85.220.176] (HELO mail-vc0-f176.google.com) (209.85.220.176) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 10 Sep 2012 14:36:44 +0000 Received: by vcbfl11 with SMTP id fl11so1802173vcb.35 for ; Mon, 10 Sep 2012 07:36:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type; bh=frA5/i2dV1YHXRzoH2OO9AjRdfqDDuIT7mFzlWs9Aw8=; b=oZ98nCAcLi6CIP8/YJLVw/oCeAB7R4oI0Nm3JXKFcpkyTwzzq90DBxq2HdcZYxay2w dbVI/1et6CF5fAGo5W4HoYWcPqZhvLBmYaYf6DrUsil75kD0PPFIniiREmIAlASthZsN QLiCDd34sqWUkAxrummI9CkBRf+5bSoH9qhZTfL0NbX7OEP9D6K5ZTToGXn4YDaWenl8 g3nyH676NFeNUHyVNGSTbPJfYDCZr8btE2uCORVceKXY/zv1M34fbqUbzWHibHxCwTHV +w1JueqaenLx3vAM8UxsSDAalQLnOHJcjsRR+EMa4dHcD3bxGUw31TUCiDRKRSYKXW1S D4WQ== Received: by 10.58.247.165 with SMTP id yf5mr21275719vec.50.1347287783781; Mon, 10 Sep 2012 07:36:23 -0700 (PDT) MIME-Version: 1.0 Received: by 10.220.239.14 with HTTP; Mon, 10 Sep 2012 07:36:02 -0700 (PDT) In-Reply-To: References: From: Robert Muir Date: Mon, 10 Sep 2012 10:36:02 -0400 Message-ID: Subject: Re: Purpose of CachingCollector.create(boolean, boolean, double) ? To: dev@lucene.apache.org Content-Type: text/plain; charset=UTF-8 X-Virus-Checked: Checked by ClamAV on apache.org fwiw, this method is only used by tests. I think we could safely remove it and just have those tests (TestCachingCollector and one grouping test) provide whatever collector they need instead. On Mon, Sep 10, 2012 at 10:33 AM, Robert Muir wrote: > I'm looking at this code and a little confused: > > this wraps a no-op collector, so what happens if the maxRAMMB is > exceeded? It seems to me in that case those hits would just be > silently dropped?! > > Is this intentional? -- lucidworks.com --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org For additional commands, e-mail: dev-help@lucene.apache.org