Lucene - Core
  1. Lucene - Core
  2. LUCENE-2278

FastVectorHighlighter: highlighted term is out of alignment in multi-valued NOT_ANALYZED field

    Details

    • Lucene Fields:
      New
    1. LUCENE-2278.patch
      8 kB
      Koji Sekiguchi

      Activity

      Koji Sekiguchi created issue -
      Hide
      Koji Sekiguchi added a comment -

      Fix and a test case attached.

      Show
      Koji Sekiguchi added a comment - Fix and a test case attached.
      Koji Sekiguchi made changes -
      Field Original Value New Value
      Attachment LUCENE-2278.patch [ 12436596 ]
      Hide
      Digy added a comment -

      Besides passing docid+fieldname to FVH, can there be another method that accepts the original text to be highlighted?
      like:

      	getBestFragment( final FieldQuery fieldQuery, IndexReader reader, String originalText, int fragCharSize ) 
      

      So, the field content can be stored outside of lucene resulting in a smaller index.

      DIGY

      Show
      Digy added a comment - Besides passing docid+fieldname to FVH, can there be another method that accepts the original text to be highlighted? like: getBestFragment( final FieldQuery fieldQuery, IndexReader reader, String originalText, int fragCharSize ) So, the field content can be stored outside of lucene resulting in a smaller index. DIGY
      Hide
      Koji Sekiguchi added a comment -

      getBestFragment() method needs docId and fieldName in order to get a TermPositionVector object from IndexReader when trying to make snippets.

      FVH could be acceptable an array of String so that it can make snippets from String[] outside of Lucene, rather than stored values. But it is out of this thread.

      Show
      Koji Sekiguchi added a comment - getBestFragment() method needs docId and fieldName in order to get a TermPositionVector object from IndexReader when trying to make snippets. FVH could be acceptable an array of String so that it can make snippets from String[] outside of Lucene, rather than stored values. But it is out of this thread.
      Hide
      Koji Sekiguchi added a comment -

      I'll commit in a few days.

      Show
      Koji Sekiguchi added a comment - I'll commit in a few days.
      Hide
      Koji Sekiguchi added a comment -

      Committed revision 916090.

      Show
      Koji Sekiguchi added a comment - Committed revision 916090.
      Koji Sekiguchi made changes -
      Status Open [ 1 ] Resolved [ 5 ]
      Resolution Fixed [ 1 ]
      Hide
      Robert Muir added a comment -

      reopening for possible 2.9.4/3.0.3 backport.

      Show
      Robert Muir added a comment - reopening for possible 2.9.4/3.0.3 backport.
      Robert Muir made changes -
      Resolution Fixed [ 1 ]
      Status Resolved [ 5 ] Reopened [ 4 ]
      Robert Muir made changes -
      Fix Version/s 2.9.4 [ 12315148 ]
      Fix Version/s 3.0.3 [ 12315147 ]
      Fix Version/s 3.1 [ 12314822 ]
      Hide
      Koji Sekiguchi added a comment -

      Committed revision 1028833(2.9), 1028839(3.0).

      Show
      Koji Sekiguchi added a comment - Committed revision 1028833(2.9), 1028839(3.0).
      Koji Sekiguchi made changes -
      Status Reopened [ 4 ] Resolved [ 5 ]
      Assignee Koji Sekiguchi [ koji ]
      Resolution Fixed [ 1 ]
      Uwe Schindler made changes -
      Status Resolved [ 5 ] Closed [ 6 ]
      Mark Thomas made changes -
      Workflow jira [ 12499744 ] Default workflow, editable Closed status [ 12563773 ]
      Mark Thomas made changes -
      Workflow Default workflow, editable Closed status [ 12563773 ] jira [ 12585315 ]

        People

        • Assignee:
          Koji Sekiguchi
          Reporter:
          Koji Sekiguchi
        • Votes:
          0 Vote for this issue
          Watchers:
          2 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development