Thanks for the correction, you're quite correct. And if we move forward with more query time config, we can reuse the same syntax
On Mon, Nov 27, 2017 at 10:27 PM dsmiley <[email protected]> wrote: > Github user dsmiley commented on a diff in the pull request: > > https://github.com/apache/lucene-solr/pull/275#discussion_r153387514 > > --- Diff: solr/core/src/java/org/apache/solr/schema/FieldType.java --- > @@ -905,6 +905,7 @@ protected void checkSupportsDocValues() { > protected static final String ENABLE_GRAPH_QUERIES = > "enableGraphQueries"; > private static final String ARGS = "args"; > private static final String POSITION_INCREMENT_GAP = > "positionIncrementGap"; > + protected static final String SCORE_OVERLAPS = "scoreOverlaps"; > --- End diff -- > > I need to correct you one one point: Solr has had a syntax for > per-field query parameters for a long time. The syntax is > `f.fieldName.parameterName` e.g. `f.title.hl.snippets` SolrJ's > SolrParams has convenience methods for this on the implementation side. > Perhaps you overlooked this because most users only use it in the context > of faceting parameters, even though it's certainly not unique to faceting > (as in the example above for highlighting). I'm not aware of any query > parser that uses it yet but they certainly could. > > Any way, I suppose even if we agree we'd like some query time > customizability of this (and other settings), it would still be nice to > establish a default fallback on the FieldType. > > > --- > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > > -- Consultant, OpenSource Connections. Contact info at http://o19s.com/about-us/doug-turnbull/; Free/Busy (http://bit.ly/dougs_cal)
