Return-Path: Delivered-To: apmail-lucene-java-dev-archive@www.apache.org Received: (qmail 78510 invoked from network); 13 Oct 2009 07:44:08 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 13 Oct 2009 07:44:08 -0000 Received: (qmail 29228 invoked by uid 500); 13 Oct 2009 07:44:07 -0000 Delivered-To: apmail-lucene-java-dev-archive@lucene.apache.org Received: (qmail 29149 invoked by uid 500); 13 Oct 2009 07:44:07 -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 29136 invoked by uid 99); 13 Oct 2009 07:44:07 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 13 Oct 2009 07:44:07 +0000 X-ASF-Spam-Status: No, hits=1.2 required=10.0 tests=SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (nike.apache.org: local policy) Received: from [85.25.71.29] (HELO mail.troja.net) (85.25.71.29) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 13 Oct 2009 07:43:55 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by mail.troja.net (Postfix) with ESMTP id DEE4CD36004 for ; Tue, 13 Oct 2009 09:43:34 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at mail.troja.net Received: from mail.troja.net ([127.0.0.1]) by localhost (megaira.troja.net [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id qSryyLHE0PbT for ; Tue, 13 Oct 2009 09:43:26 +0200 (CEST) Received: from VEGA (port-83-236-62-3.dynamic.qsc.de [83.236.62.3]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.troja.net (Postfix) with ESMTPSA id 76E47D36003 for ; Tue, 13 Oct 2009 09:43:26 +0200 (CEST) From: "Uwe Schindler" To: References: <20091013065940.CF6B72388904@eris.apache.org> <77964A755346425797B1F809C4514B6C@VEGA> <4AD42B7A.5000902@gmail.com> <4AD42C94.2060608@gmail.com> Subject: RE: svn commit: r824611 - in /lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/lucene/search/spans: FieldMaskingSpanQuery.java SpanFirstQuery.java SpanNearQuery.java SpanNotQuery.java SpanOrQuery.java Date: Tue, 13 Oct 2009 09:43:27 +0200 Message-ID: <4E8D265BF2904BAFBC9FF63EE8BC5039@VEGA> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Office Outlook 11 In-Reply-To: <4AD42C94.2060608@gmail.com> Thread-Index: AcpL1yhSWX40yxQZQ4C9NbjnQDfbcgAAPD4Q X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.5579 X-Virus-Checked: Checked by ClamAV on apache.org Yes, thats why we do the tests. By this it is possible to test compiled Java 1.4 code against new Java 1.5 lucene core with generics and test, that no upper generics boundaries (e.g. by things like ) are violated. But if you rewrite the tests to only use the API of lucene 3.0 and no deprecated methods it should pass and it has no effect, if an additional deprecated method is still available in the branch's code. If we have to remove all deprecated code also from the backwards branch, we would not need the branch at all. So this commit is definitely not needed (and I tested it, it works without). In the backwards branch we should only fix the tests, never the core code. If we do it, it is contra-productive. There were some edge cases, when we have backwards-incompatible changes in 2.9. But this is definitely not a backwards break. ----- Uwe Schindler H.-H.-Meier-Allee 63, D-28213 Bremen http://www.thetaphi.de eMail: uwe@thetaphi.de > -----Original Message----- > From: Michael Busch [mailto:buschmic@gmail.com] > Sent: Tuesday, October 13, 2009 9:30 AM > To: java-dev@lucene.apache.org > Subject: Re: svn commit: r824611 - in > /lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luc > ene/search/spans: FieldMaskingSpanQuery.java SpanFirstQuery.java > SpanNearQuery.java SpanNotQuery.java SpanOrQuery.java > > Yes that's indeed the case, see LUCENE-1529. > > Michael > > On 10/13/09 12:25 AM, Michael Busch wrote: > > It was weird - I ran all the tests before I did the previous commit > > and it worked fine. Then after committing I wanted to doublecheck by > > running 'ant test-tag' and got the compile errors. > > > > I think something is wrong with my eclipse and/or svn. But I also > > switched from tortoise to command-line recently - so maybe I'm just > > clumsy. Anyway, the new tag is working now, sorry for the noise. > > > > To your question: Wasn't there a fix recently to test-tag to test > > drop-in backwards-compatibility? Which means that it compiles the > > tests first against the sources of the back-compat branch, but then > > runs them against the new trunk JAR? That's why this commit is > > necessary I think. > > > > Michael > > > > On 10/13/09 12:18 AM, Uwe Schindler wrote: > >> I wonder why this commit is needed. It only affects the core classes, > >> not th > >> tests. To compile correct backwards tests it should not be important > >> if the > >> methods exist or not. > >> > >> ----- > >> Uwe Schindler > >> H.-H.-Meier-Allee 63, D-28213 Bremen > >> http://www.thetaphi.de > >> eMail: uwe@thetaphi.de > >> > >> > >>> -----Original Message----- > >>> From: buschmi@apache.org [mailto:buschmi@apache.org] > >>> Sent: Tuesday, October 13, 2009 9:00 AM > >>> To: java-commits@lucene.apache.org > >>> Subject: svn commit: r824611 - in > >>> > /lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luc > >>> > >>> ene/search/spans: FieldMaskingSpanQuery.java SpanFirstQuery.java > >>> SpanNearQuery.java SpanNotQuery.java SpanOrQuery.java > >>> > >>> Author: buschmi > >>> Date: Tue Oct 13 06:59:40 2009 > >>> New Revision: 824611 > >>> > >>> URL: http://svn.apache.org/viewvc?rev=824611&view=rev > >>> Log: > >>> More fixes that were accidentially left out in the previous commit > >>> > >>> Modified: > >>> > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/FieldMaskingSpanQuery.java > >>> > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanFirstQuery.java > >>> > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanNearQuery.java > >>> > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanNotQuery.java > >>> > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanOrQuery.java > >>> > >>> Modified: > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/FieldMaskingSpanQuery.java > >>> URL: > >>> > http://svn.apache.org/viewvc/lucene/java/branches/lucene_2_9_back_compat_t > >>> > >>> > ests/src/java/org/apache/lucene/search/spans/FieldMaskingSpanQuery.java?re > >>> > >>> v=824611&r1=824610&r2=824611&view=diff > >>> > ========================================================================== > >>> > >>> ==== > >>> --- > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/FieldMaskingSpanQuery.java (original) > >>> +++ > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/FieldMaskingSpanQuery.java Tue Oct 13 06:59:40 2009 > >>> @@ -94,11 +94,6 @@ > >>> return maskedQuery.getSpans(reader); > >>> } > >>> > >>> - /** @deprecated use {@link #extractTerms(Set)} instead. */ > >>> - public Collection getTerms() { > >>> - return maskedQuery.getTerms(); > >>> - } > >>> - > >>> public void extractTerms(Set terms) { > >>> maskedQuery.extractTerms(terms); > >>> } > >>> > >>> Modified: > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanFirstQuery.java > >>> URL: > >>> > http://svn.apache.org/viewvc/lucene/java/branches/lucene_2_9_back_compat_t > >>> > >>> > ests/src/java/org/apache/lucene/search/spans/SpanFirstQuery.java?rev=82461 > >>> > >>> 1&r1=824610&r2=824611&view=diff > >>> > ========================================================================== > >>> > >>> ==== > >>> --- > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanFirstQuery.java (original) > >>> +++ > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanFirstQuery.java Tue Oct 13 06:59:40 2009 > >>> @@ -47,12 +47,6 @@ > >>> > >>> public String getField() { return match.getField(); } > >>> > >>> - /** Returns a collection of all terms matched by this query. > >>> - * @deprecated use extractTerms instead > >>> - * @see #extractTerms(Set) > >>> - */ > >>> - public Collection getTerms() { return match.getTerms(); } > >>> - > >>> public String toString(String field) { > >>> StringBuffer buffer = new StringBuffer(); > >>> buffer.append("spanFirst("); > >>> > >>> Modified: > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanNearQuery.java > >>> URL: > >>> > http://svn.apache.org/viewvc/lucene/java/branches/lucene_2_9_back_compat_t > >>> > >>> > ests/src/java/org/apache/lucene/search/spans/SpanNearQuery.java?rev=824611 > >>> > >>> &r1=824610&r2=824611&view=diff > >>> > ========================================================================== > >>> > >>> ==== > >>> --- > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanNearQuery.java (original) > >>> +++ > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanNearQuery.java Tue Oct 13 06:59:40 2009 > >>> @@ -80,20 +80,6 @@ > >>> > >>> public String getField() { return field; } > >>> > >>> - /** Returns a collection of all terms matched by this query. > >>> - * @deprecated use extractTerms instead > >>> - * @see #extractTerms(Set) > >>> - */ > >>> - public Collection getTerms() { > >>> - Collection terms = new ArrayList(); > >>> - Iterator i = clauses.iterator(); > >>> - while (i.hasNext()) { > >>> - SpanQuery clause = (SpanQuery)i.next(); > >>> - terms.addAll(clause.getTerms()); > >>> - } > >>> - return terms; > >>> - } > >>> - > >>> public void extractTerms(Set terms) { > >>> Iterator i = clauses.iterator(); > >>> while (i.hasNext()) { > >>> > >>> Modified: > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanNotQuery.java > >>> URL: > >>> > http://svn.apache.org/viewvc/lucene/java/branches/lucene_2_9_back_compat_t > >>> > >>> > ests/src/java/org/apache/lucene/search/spans/SpanNotQuery.java?rev=824611& > >>> > >>> r1=824610&r2=824611&view=diff > >>> > ========================================================================== > >>> > >>> ==== > >>> --- > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanNotQuery.java (original) > >>> +++ > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanNotQuery.java Tue Oct 13 06:59:40 2009 > >>> @@ -49,12 +49,6 @@ > >>> > >>> public String getField() { return include.getField(); } > >>> > >>> - /** Returns a collection of all terms matched by this query. > >>> - * @deprecated use extractTerms instead > >>> - * @see #extractTerms(Set) > >>> - */ > >>> - public Collection getTerms() { return include.getTerms(); } > >>> - > >>> public void extractTerms(Set terms) { > >>> include.extractTerms(terms); } > >>> > >>> public String toString(String field) { > >>> > >>> Modified: > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanOrQuery.java > >>> URL: > >>> > http://svn.apache.org/viewvc/lucene/java/branches/lucene_2_9_back_compat_t > >>> > >>> > ests/src/java/org/apache/lucene/search/spans/SpanOrQuery.java?rev=824611&r > >>> > >>> 1=824610&r2=824611&view=diff > >>> > ========================================================================== > >>> > >>> ==== > >>> --- > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanOrQuery.java (original) > >>> +++ > >>> > lucene/java/branches/lucene_2_9_back_compat_tests/src/java/org/apache/luce > >>> > >>> ne/search/spans/SpanOrQuery.java Tue Oct 13 06:59:40 2009 > >>> @@ -58,20 +58,6 @@ > >>> > >>> public String getField() { return field; } > >>> > >>> - /** Returns a collection of all terms matched by this query. > >>> - * @deprecated use extractTerms instead > >>> - * @see #extractTerms(Set) > >>> - */ > >>> - public Collection getTerms() { > >>> - Collection terms = new ArrayList(); > >>> - Iterator i = clauses.iterator(); > >>> - while (i.hasNext()) { > >>> - SpanQuery clause = (SpanQuery)i.next(); > >>> - terms.addAll(clause.getTerms()); > >>> - } > >>> - return terms; > >>> - } > >>> - > >>> public void extractTerms(Set terms) { > >>> Iterator i = clauses.iterator(); > >>> while (i.hasNext()) { > >>> > >> > >> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org > >> For additional commands, e-mail: java-dev-help@lucene.apache.org > >> > >> > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org > For additional commands, e-mail: java-dev-help@lucene.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org For additional commands, e-mail: java-dev-help@lucene.apache.org