You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-dev@lucene.apache.org by "Lars Kotthoff (JIRA)" <ji...@apache.org> on 2008/04/15 07:37:06 UTC
[jira] Created: (SOLR-540) Add support for hl.fl=*
Add support for hl.fl=*
-----------------------
Key: SOLR-540
URL: https://issues.apache.org/jira/browse/SOLR-540
Project: Solr
Issue Type: New Feature
Components: highlighter
Affects Versions: 1.3
Environment: Tomcat 5.5
Reporter: Lars Kotthoff
Priority: Minor
Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: SOLR-540-highlight-all.patch
The old patch didn't verify properly that only stored static fields are returned. The new patch fixes this.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12659702#action_12659702 ]
Lars Kotthoff commented on SOLR-540:
------------------------------------
Ah, just remembered, see SOLR-650.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Fix For: 1.4
>
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540.patch, SOLR-540.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "Hoss Man (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12589160#action_12589160 ]
Hoss Man commented on SOLR-540:
-------------------------------
a quick glance at this patch indicates that it has the same characteristics that concerned me about SOLR-247 ...
bq. with somehting like {{fl=\*}}, we're only talking about stored fields ... storing a field makes no sense unless you plan on returning it in the field list some of the time, so {{fl=\*}} makes sense as a "return all of hte fields that are possible to return" option.
... this patch attempts to highlight every field in the index, stored or otherwise, termVectors or otherwise.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: solr-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: solr-highlight-all.patch
Patch as described against svn revision 648118.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: solr-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12641726#action_12641726 ]
Lars Kotthoff commented on SOLR-540:
------------------------------------
Hmm, actually the list of fields to highlight on should be rebuilt in cases like that. Are you opening a new searcher for the new index?
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: (was: solr-highlight-all.patch)
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: solr-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "David Smiley (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12641672#action_12641672 ]
David Smiley commented on SOLR-540:
-----------------------------------
I've seen cases where I'd get an error because the previous index contained a field that is no longer in the current schema. I've corrected this by Changing line 171 of SolrIndexSearcher to go from a simple lookup of the field to one that catches this exception like so:
{code}
SchemaField field;
try {
field = schema.getField(fieldName);
} catch (SolrException e) {
log.fine("Skipping indexed field not found in schema e:"+e);//no reason to log stacktrace
continue;
}
{code}
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: SOLR-540-highlight-all.patch
Syncing patch with trunk.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "Yonik Seeley (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12659254#action_12659254 ]
Yonik Seeley commented on SOLR-540:
-----------------------------------
Looks useful.
It's questionable if we always want to generate the fields collection, or if we want to defer until first requested, but it's an easy change to do it later if we decide to (i.e. not an interface change).
Perhaps the test for text fields should use istanceof instead of class comparison so subclasses of TextField will work?
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "Yonik Seeley (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12659685#action_12659685 ]
Yonik Seeley commented on SOLR-540:
-----------------------------------
I imagine it would be useful to be able to use globbing in conjunction with normal field specification, or multiple globs?
example: hl.fl=body,*_text,*_misc
I didn't hold up committing this issue for that since it would be a strict API superset.
Thoughts?
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Fix For: 1.4
>
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540.patch, SOLR-540.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: SOLR-540.patch
Syncing patch with trunk.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12589384#action_12589384 ]
Lars Kotthoff commented on SOLR-540:
------------------------------------
Erik, I agree. This issue has been around for almost a year now though. The general case is much harder to implement and get right than such a apecific one; therefore it might be better to start by implementing it for something not too difficult (like highlighting) and see what people think of it. First to see which features are really needed, but also to see whether it's the right approach at all.
Chris, good point. I didn't think about this because in the schema I'm using with the patched Solr instance all fields happen to be stored. I've changed the patch to return only stored fields. Now, this is arguably not the right place for generic give-me-all-stored-fields code and the implementation is inefficient. My knowledge of the Solr code base isn't good enough though to figure out the right place for this code -- it needs to talk to both Solr and Lucene code to get all static and dynamic fields. Should this maybe go into the SolrCore code? It could maintain a list of stored fields which is assembled on startup and updated on <commit/> instead of reassembling it for every query.
Note that the latest version of the patch could also easily be tweaked to allow arbitrary wildcard specifications of the fields to highlight (e.g. hl.fl=text_*).
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: solr-highlight-all.patch, solr-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12642272#action_12642272 ]
Lars Kotthoff commented on SOLR-540:
------------------------------------
I've run the tests repeatedly and wasn't able to reproduce the problem. Can you perhaps nail it down to a particular test?
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "David Smiley (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12627985#action_12627985 ]
David Smiley commented on SOLR-540:
-----------------------------------
Hey SOLR-540 people, please see the comment thread on SOLR-750 which is apparently a bug with this patch.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (SOLR-540) Add support for hl.fl=*
Posted by "Yonik Seeley (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Yonik Seeley resolved SOLR-540.
-------------------------------
Resolution: Fixed
Fix Version/s: 1.4
Committed, thanks Lars!
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Fix For: 1.4
>
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540.patch, SOLR-540.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: SOLR-540-highlight-all.patch
I've changed a couple of things and uploaded the new patch.
* The stored fields are no longer determined in SolrHighlighter for every request, but in SolrIndexSearcher when creating a new object. The implementation now simply queries the schema for all field names the index reader knows about and checks for each whether it's stored. The collections of field names and stored field names are exposed through getters. SolrHighlighter simply matches against the items of the storedFieldNames collection.
* Added generic support for wildcards; in addition to "*" things like "foo_*" also work now. Now for example the prototype of a dynamic field may be given as field name to highlight on and all instances of that prototype will be highlighted.
* Added a unit test to verify that highlighting with wildcard fields works as expected.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: SOLR-540-highlight-all.patch
Attaching new patch which only highlights on stored text/string fields. Added test case to verify that.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: SOLR-540.patch
Attaching new patch which uses instanceof instead of comparing the classes directly and defers the generation of the list of fields to highlight on until they're requested. I left the assignment of fields the index reader knows about in the constructor because this shouldn't be too expensive. It could easily be moved into the getter as well though.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540.patch, SOLR-540.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12659700#action_12659700 ]
Lars Kotthoff commented on SOLR-540:
------------------------------------
Muchos gracias.
Re globbing, I'm sure I saw an issue about doing that in general somewhere. Can't find it though, does anybody remember? Anyway, I think this should be tackled more generally, i.e. have a utility function that takes a list of parameters (possibly containing globs) and a list of possible values and returns the matching values. Most likely this function would be declared in an interface somewhere and implemented differently for highlighting, faceting, etc. This would be quite a major change though and should definitely be a separate issue.
(Yonik, next on my list is SOLR-634 btw. HINT HINT) ;)
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Fix For: 1.4
>
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540.patch, SOLR-540.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: solr-highlight-all.patch
New patch returning only the stored fields.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: solr-highlight-all.patch, solr-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: SOLR-540-highlight-all.patch
Attaching patch which handles the case where a field is present in the index but not defined in the field by logging a warning instead of throwing a RuntimeException.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "David Smiley (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12627071#action_12627071 ]
David Smiley commented on SOLR-540:
-----------------------------------
I'd like this incorporated into Solr. I stupidly didn't search for this feature and I went and did it myself... was just about to submit my own patch. Doh!
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: SOLR-540-highlight-all.patch
Syncing patch with trunk.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (SOLR-540) Add support for hl.fl=*
Posted by "Lars Kotthoff (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lars Kotthoff updated SOLR-540:
-------------------------------
Attachment: (was: solr-highlight-all.patch)
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "David Smiley (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12641993#action_12641993 ]
David Smiley commented on SOLR-540:
-----------------------------------
This happens to me when I run the ant unit tests on a few of the tests ( I don't have an example off hand ). It may be related to re-running tests without cleaning; I'm not sure. Perhaps the state of past-runs are affected.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch, SOLR-540-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-540) Add support for hl.fl=*
Posted by "Erik Hatcher (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12589047#action_12589047 ]
Erik Hatcher commented on SOLR-540:
-----------------------------------
While this is a useful addition, should we tackle this as part of the a more general field aliasing/globbing feature? See http://wiki.apache.org/solr/FieldAliasesAndGlobsInParams for more details.
> Add support for hl.fl=*
> -----------------------
>
> Key: SOLR-540
> URL: https://issues.apache.org/jira/browse/SOLR-540
> Project: Solr
> Issue Type: New Feature
> Components: highlighter
> Affects Versions: 1.3
> Environment: Tomcat 5.5
> Reporter: Lars Kotthoff
> Priority: Minor
> Attachments: solr-highlight-all.patch
>
>
> Adds support for the star value for the hl.fl parameter, i.e. highlighting will be done on all fields (static and dynamic). Particularly useful in conjunction with hl.requireFieldMatch=true, this way one can specify "generic" highlighting parameters independent of the query/searched fields.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.