Robert Muir created LUCENE-5231:
-----------------------------------

             Summary: better interoperability of expressions/ with valuesource
                 Key: LUCENE-5231
                 URL: https://issues.apache.org/jira/browse/LUCENE-5231
             Project: Lucene - Core
          Issue Type: Task
            Reporter: Robert Muir
         Attachments: LUCENE-5231.patch

A few things i noticed, while trying to work on e.g. integration of this with 
solr and just playing around:

* No way for a custom Bindings to currently bind the score, as the necessary 
stuff is package private. This adds a simple protected method to Bindings to 
enable this.
* Expression.getValueSource() cannot in general be used easily by other things 
(e.g. interoperate with function queries and so on), because it expects you 
pass it this "custom cache". This is an impl detail, its easy to remove this 
restriction and still compute subs only once.
* if you try to bind the score and don't have the scorer setup, you should get 
a clear exception: not NPE.
* Each binding is looked up per-segment, which is bad. we should minimize the 
lookups to only in the CTOR.
* This makes validation considerably simpler and less error-prone, so easy that 
I don't think we need it in the base class either, I moved this to just a 
simple helper method on SimpleBindings. It also found a bug in the equals() 
test.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to