Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.9
    • Fix Version/s: 2.9
    • Component/s: modules/other
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      We should rename the new QueryParser so it's clearer that it's
      Lucene's default QueryParser, going forward, and not just a temporary
      "bridge" to a future new QueryParser.

      How about we rename oal.queryParser.original -->
      oal.queryParser.standard (can't use "default": it's a Java keyword)?
      Then, leave the OriginalQueryParserHelper under that package, but
      simply rename it to QueryParser?

      This way if we create other sub-packages in the future, eg
      ComplexPhraseQueryParser, they too can have a QueryParser class under
      them, to make it clear that's the "top" class you use to parse
      queries.

      1. LUCENE-1782.patch
        848 kB
        Michael McCandless
      2. LUCENE-1782.patch
        823 kB
        Michael McCandless

        Issue Links

          Activity

          Michael McCandless created issue -
          Michael McCandless made changes -
          Field Original Value New Value
          Attachment LUCENE-1782.patch [ 12415644 ]
          Michael McCandless made changes -
          Attachment LUCENE-1782.patch [ 12415717 ]
          Luis Alves made changes -
          Link This issue depends on LUCENE-1567 [ LUCENE-1567 ]
          Michael McCandless made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Mark Miller made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Mark Thomas made changes -
          Workflow jira [ 12472360 ] Default workflow, editable Closed status [ 12562917 ]
          Mark Thomas made changes -
          Workflow Default workflow, editable Closed status [ 12562917 ] jira [ 12583799 ]
          Gavin made changes -
          Link This issue depends on LUCENE-1567 [ LUCENE-1567 ]
          Gavin made changes -
          Link This issue depends upon LUCENE-1567 [ LUCENE-1567 ]

            People

            • Assignee:
              Michael McCandless
              Reporter:
              Michael McCandless
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development