Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0-ALPHA
    • Component/s: modules/queryparser
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      Each of the QueryParsers will be ported across.

      Those which use the flexible parsing framework will be placed under the package flexible. The StandardQueryParser will be renamed to FlexibleQueryParser and surround.QueryParser will be renamed to SurroundQueryParser.

      1. LUCENE-3285.patch
        44 kB
        Chris Male
      2. LUCENE-3285-flexible.patch
        265 kB
        Chris Male

        Activity

        Hide
        Chris Male added a comment -

        Patch which moves the AnalyzingQueryParser, ComplexPhraseQueryParser and the Surround QueryParser to the new queryparser module.

        This leaves just the flexible queryparser in the contrib.

        Every compiles, tests pass. I'm going to commit this soon and then move onto the flexible queryparser. Naming of the QPs themselves will be addressed afterwards.

        Show
        Chris Male added a comment - Patch which moves the AnalyzingQueryParser, ComplexPhraseQueryParser and the Surround QueryParser to the new queryparser module. This leaves just the flexible queryparser in the contrib. Every compiles, tests pass. I'm going to commit this soon and then move onto the flexible queryparser. Naming of the QPs themselves will be addressed afterwards.
        Hide
        Chris Male added a comment -

        Command for patch:

        # AnalyzingQueryParser
        
        svn --parents move lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/analyzing/* modules/queryparser/src/java/org/apache/lucene/queryparser/analyzing/
        svn --parents move lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/analyzing/TestAnalyzingQueryParser.java modules/queryparser/src/test/org/apache/lucene/queryparser/analyzing/TestAnalyzingQueryParser.java
        svn delete lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/analyzing
        svn delete lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/analyzing
        
        # ComplexPhraseQueryParser
        
        svn --parents move lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/complexPhrase/* modules/queryparser/src/java/org/apache/lucene/queryparser/complexPhrase/
        svn --parents move lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/complexPhrase/TestComplexPhraseQuery.java modules/queryparser/src/test/org/apache/lucene/queryparser/complexPhrase/TestComplexPhraseQuery.java
        svn delete lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/complexPhrase
        svn delete lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/complexPhrase
        
        # ExtendableQueryParser
        
        svn --parents move lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/ext/* modules/queryparser/src/java/org/apache/lucene/queryparser/ext/
        svn --parents move lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/ext/* modules/queryparser/src/test/org/apache/lucene/queryparser/ext/
        svn delete lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/ext
        svn delete lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/ext
        
        # SurroundQueryParser
        
        svn --parents move lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/surround/* modules/queryparser/src/java/org/apache/lucene/queryparser/surround/
        svn --parents move lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/surround/query/* modules/queryparser/src/test/org/apache/lucene/queryparser/surround/query/
        svn delete lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/surround
        svn delete lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/surround
        
        Show
        Chris Male added a comment - Command for patch: # AnalyzingQueryParser svn --parents move lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/analyzing/* modules/queryparser/src/java/org/apache/lucene/queryparser/analyzing/ svn --parents move lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/analyzing/TestAnalyzingQueryParser.java modules/queryparser/src/test/org/apache/lucene/queryparser/analyzing/TestAnalyzingQueryParser.java svn delete lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/analyzing svn delete lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/analyzing # ComplexPhraseQueryParser svn --parents move lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/complexPhrase/* modules/queryparser/src/java/org/apache/lucene/queryparser/complexPhrase/ svn --parents move lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/complexPhrase/TestComplexPhraseQuery.java modules/queryparser/src/test/org/apache/lucene/queryparser/complexPhrase/TestComplexPhraseQuery.java svn delete lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/complexPhrase svn delete lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/complexPhrase # ExtendableQueryParser svn --parents move lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/ext/* modules/queryparser/src/java/org/apache/lucene/queryparser/ext/ svn --parents move lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/ext/* modules/queryparser/src/test/org/apache/lucene/queryparser/ext/ svn delete lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/ext svn delete lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/ext # SurroundQueryParser svn --parents move lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/surround/* modules/queryparser/src/java/org/apache/lucene/queryparser/surround/ svn --parents move lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/surround/query/* modules/queryparser/src/test/org/apache/lucene/queryparser/surround/query/ svn delete lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/surround svn delete lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/surround
        Hide
        Chris Male added a comment -

        Committed revision 1145430.

        Now moving onto flexible QP.

        Show
        Chris Male added a comment - Committed revision 1145430. Now moving onto flexible QP.
        Hide
        Chris Male added a comment -

        Patch which moves the flexible queryparser to the new module.

        Removes contrib/queryparser as well.

        Show
        Chris Male added a comment - Patch which moves the flexible queryparser to the new module. Removes contrib/queryparser as well.
        Hide
        Chris Male added a comment -

        Command for flexible patch:

        svn --parents move lucene/contrib/queryparser/README.txt modules/queryparser/docs/surround/README.txt
        svn --parents move lucene/contrib/queryparser/README.javacc modules/queryparser/docs/flexible/README.javacc
        svn --parents move lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/* modules/queryparser/src/java/org/apache/lucene/queryparser/flexible/
        svn --parents move lucene/contrib/queryparser/src/resources/org/apache/lucene/queryParser/core/messages/QueryParserMessages.properties modules/queryparser/src/resources/org/apache/lucene/queryparser/flexible/core/messages/QueryParserMessages.properties
        svn --parents move lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/* modules/queryparser/src/test/org/apache/lucene/queryparser/flexible/
        svn move lucene/contrib/queryparser/src/java/overview.html modules/queryparser/src/java/org/apache/lucene/queryparser/flexible/package.html
        svn delete lucene/contrib/queryparser
        svn delete dev-tools/maven/lucene/contrib/queryparser
        svn delete dev-tools/idea/lucene/contrib/queryparser
        
        Show
        Chris Male added a comment - Command for flexible patch: svn --parents move lucene/contrib/queryparser/README.txt modules/queryparser/docs/surround/README.txt svn --parents move lucene/contrib/queryparser/README.javacc modules/queryparser/docs/flexible/README.javacc svn --parents move lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/* modules/queryparser/src/java/org/apache/lucene/queryparser/flexible/ svn --parents move lucene/contrib/queryparser/src/resources/org/apache/lucene/queryParser/core/messages/QueryParserMessages.properties modules/queryparser/src/resources/org/apache/lucene/queryparser/flexible/core/messages/QueryParserMessages.properties svn --parents move lucene/contrib/queryparser/src/test/org/apache/lucene/queryParser/* modules/queryparser/src/test/org/apache/lucene/queryparser/flexible/ svn move lucene/contrib/queryparser/src/java/overview.html modules/queryparser/src/java/org/apache/lucene/queryparser/flexible/ package .html svn delete lucene/contrib/queryparser svn delete dev-tools/maven/lucene/contrib/queryparser svn delete dev-tools/idea/lucene/contrib/queryparser
        Hide
        Chris Male added a comment -

        Committed revision 1146548.

        Thank god.

        Show
        Chris Male added a comment - Committed revision 1146548. Thank god.

          People

          • Assignee:
            Chris Male
            Reporter:
            Chris Male
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development