Return-Path: Delivered-To: apmail-lucene-java-dev-archive@www.apache.org Received: (qmail 88469 invoked from network); 12 Jul 2006 19:52:56 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 12 Jul 2006 19:52:56 -0000 Received: (qmail 80554 invoked by uid 500); 12 Jul 2006 19:52:53 -0000 Delivered-To: apmail-lucene-java-dev-archive@lucene.apache.org Received: (qmail 80522 invoked by uid 500); 12 Jul 2006 19:52:53 -0000 Mailing-List: contact java-dev-help@lucene.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: java-dev@lucene.apache.org Delivered-To: mailing list java-dev@lucene.apache.org Received: (qmail 80510 invoked by uid 99); 12 Jul 2006 19:52:53 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 12 Jul 2006 12:52:53 -0700 X-ASF-Spam-Status: No, hits=0.5 required=10.0 tests=DNS_FROM_RFC_ABUSE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: domain of yseeley@gmail.com designates 64.233.166.180 as permitted sender) Received: from [64.233.166.180] (HELO py-out-1112.google.com) (64.233.166.180) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 12 Jul 2006 12:52:52 -0700 Received: by py-out-1112.google.com with SMTP id z59so370613pyg for ; Wed, 12 Jul 2006 12:52:32 -0700 (PDT) DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=iZF+y6Iowq/A+dTWQ9j+/gT6Z1qRAbtG+APtaIVWEOH8IbCLc1zh1aR+w+dpJvfijmblwmFQCe6EMC4UQU9j5FZ8+9gi6XJEgQP1TxfykInUgh2HWHcfLHmsQcfrXZeTzmXws69ywoZ2lwixFna6Ftn8oMfp2Wu/h2ky6EcTXUs= Received: by 10.35.121.9 with SMTP id y9mr1340651pym; Wed, 12 Jul 2006 12:52:31 -0700 (PDT) Received: by 10.35.129.12 with HTTP; Wed, 12 Jul 2006 12:52:31 -0700 (PDT) Message-ID: Date: Wed, 12 Jul 2006 15:52:31 -0400 From: "Yonik Seeley" To: java-dev@lucene.apache.org Subject: Re: [jira] Commented: (LUCENE-565) Supporting deleteDocuments in IndexWriter (Code and Performance Results Provided) In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <14645793.1152156330186.JavaMail.jira@brutus> <20060707130551.GA10773@fermat.math.technion.ac.il> X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N On 7/12/06, Ning Li wrote: > > I'll rephrase my original question: > > When implementing NewIndexModifier, what type of efficiencies do we > > get by using the new protected methods of IndexWriter vs using the > > public APIs of IndexReader and IndexWriter? > > What do you think can be best achieved by using the public APIs of > IndexReader and IndexWriter? I'm not sure I understand your question.... you mean why would one want to stick to public APIs? The IndexWriter class them is a simpler implementation that's easier to understand and verify correctness, easier to make changes+enhancements to, easier to maintain. The new class (NewIndexModifier) is not as tightly coupled and doesn't have to be inspected and fixed every time a change to IndexModifier is made. Overwriting documents (via delete then add) or interleaving adds and deletes is a common Lucene usecase... it would be a good thing if this functionality were somewhere in Lucene in an efficient form. If it can be done in a separate class, using public APIs (or at least with a minimum of protected access), without a loss in performance, then that's the way to go IMO. If you lose too much performance, then we start looking at the bare minimum functionality we need to expose to recover that performance. -Yonik http://incubator.apache.org/solr Solr, the open-source Lucene search server --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org For additional commands, e-mail: java-dev-help@lucene.apache.org