Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-822

Make FieldSelector usable from Searchable

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: None
    • Component/s: core/index, core/search
    • Labels:
      None

      Description

      Seems reasonable that you would want to be able to specify a FieldSelector from Searchable because many systems do not use IndexSearcher (where you can get a Reader), but instead use Searchable or Searcher so that Searchers and MultiSearchers can be used in a polymorphic manner.

        Activity

        Hide
        markrmiller@gmail.com Mark Miller added a comment -

        The patch misses adding the doc(int, FieldSelector) method to Searcher, which I don't think is necessary but fits with the style of the current code.

        Show
        markrmiller@gmail.com Mark Miller added a comment - The patch misses adding the doc(int, FieldSelector) method to Searcher, which I don't think is necessary but fits with the style of the current code.
        Hide
        gsingers Grant Ingersoll added a comment -

        Applied patch.

        Added unit tests to TestMultiSearcher and TestRemoteSearchable. As a result of the Remote test, had to make FieldSelector and FieldSelectorResult serializable.

        Show
        gsingers Grant Ingersoll added a comment - Applied patch. Added unit tests to TestMultiSearcher and TestRemoteSearchable. As a result of the Remote test, had to make FieldSelector and FieldSelectorResult serializable.

          People

          • Assignee:
            gsingers Grant Ingersoll
            Reporter:
            markrmiller@gmail.com Mark Miller
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development