Return-Path: Delivered-To: apmail-lucene-dev-archive@www.apache.org Received: (qmail 34711 invoked from network); 2 Dec 2010 09:45:04 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 2 Dec 2010 09:45:04 -0000 Received: (qmail 75867 invoked by uid 500); 2 Dec 2010 09:45:03 -0000 Delivered-To: apmail-lucene-dev-archive@lucene.apache.org Received: (qmail 75831 invoked by uid 500); 2 Dec 2010 09:45:02 -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 75824 invoked by uid 99); 2 Dec 2010 09:45:02 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 02 Dec 2010 09:45:02 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=10.0 tests=FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS,T_TO_NO_BRKTS_FREEMAIL X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of earwin@gmail.com designates 209.85.214.176 as permitted sender) Received: from [209.85.214.176] (HELO mail-iw0-f176.google.com) (209.85.214.176) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 02 Dec 2010 09:44:54 +0000 Received: by iwn35 with SMTP id 35so735803iwn.35 for ; Thu, 02 Dec 2010 01:44:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:content-type :content-transfer-encoding; bh=pVKwplQbsjKPGImRpP3TMWbg8AxJcQPAUBX96Wouo1Q=; b=YX4gg8k2GeMGK3LO4hmNtgyLyleFo0m9uVsYSEiqiaZCZozhvda1FW2e+lZkQi3Ta2 GqEnbRURS6VEm9vQlSzU8y8q/MXXheYYwsTvOoc+cxHIKPlvVO7tVK6n6mfU4Per+ogY h1dDtoXwodDPKfUhjwhDJv1s4MyFwpXr1cQhs= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; b=lpPmEm1mTSRuCBGPA0Tpfryaapy8ErDBU7kF5VN9xpPu1AdaZY3eplzHc9TAYBxNc9 81Pn0bDKBNKZrypDiPlkZxyFr2Dol2zZS7S5tspIQRa7ImBbBRwryjk/BGm2a3T5UNXM AzqR0N16lQkzDN15ec5LkVOCH1qcyRwYqV3Dk= MIME-Version: 1.0 Received: by 10.231.33.71 with SMTP id g7mr10318412ibd.54.1291283073276; Thu, 02 Dec 2010 01:44:33 -0800 (PST) Received: by 10.231.36.73 with HTTP; Thu, 2 Dec 2010 01:44:33 -0800 (PST) In-Reply-To: References: Date: Thu, 2 Dec 2010 12:44:33 +0300 Message-ID: Subject: Re: Consolidate MP and LMP From: Earwin Burrfoot To: dev@lucene.apache.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Virus-Checked: Checked by ClamAV on apache.org Actually, in trunk IW doesn't break on anything else. There's one private no-longer-used method I forgot to delete on my drop-all-deprecations spree. And there's a block in addIndexes, that explicitly checks instanceof, and only then casts to LMP. I'm against consolidating MP and LMP. MP is a damn interface! We should strive to make things less coupled rather than other way around. On Thu, Dec 2, 2010 at 12:25, Shai Erera wrote: > Hi > > While IndexWriter declares it accepts a general MP, it will actually fail= if > the given instance is not LogMP. So I wonder if we shouldn't consolidate > both of them into one, and pull up all of LMP features to MP. I think all= of > LMP's features are useful for any kind of MP, and if someone wants to ign= ore > them he still can. > > This is not the sort of change that fits well in trunk. IMO it can fit we= ll > in 3x too since IW didn't accept anything that is not LMP. So even if it > will appear we're breaking back-compat, we actually won't. Which is anoth= er > reason, for me, why those two should be consolidated. > > What do you think? > > Shai > --=20 Kirill Zakharenko/=D0=9A=D0=B8=D1=80=D0=B8=D0=BB=D0=BB =D0=97=D0=B0=D1=85= =D0=B0=D1=80=D0=B5=D0=BD=D0=BA=D0=BE (earwin@gmail.com) Phone: +7 (495) 683-567-4 ICQ: 104465785 --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org For additional commands, e-mail: dev-help@lucene.apache.org