[ 
https://issues.apache.org/jira/browse/SOLR-356?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12558434#action_12558434
 ] 

Doug Daniels commented on SOLR-356:
-----------------------------------

I agree that the ValueSourceParser should be isolated from the low-level 
details like grabbing commas between args.

I thought about the first option you suggested, and it seems difficult with 
recursive functions.  When the ValueSourceParser.createValueSource method 
expects another ValueSource as an arg, it would need to invoke whatever code 
created it (presumably from the FunctionQParser).   Alternately, the 
FunctionQParser could ensure that the innermost functions are run first, 
passing their completed values out to enclosing functions as params.

It seems simpler to me to go with the second option though -- locking down the 
API on FunctionQParser.  The ValueSourceParser would already have access to 
FunctionQParser, which it could call when it needs to parse a ValueSource 
argument.

What do you think?

> pluggable functions (value sources)
> -----------------------------------
>
>                 Key: SOLR-356
>                 URL: https://issues.apache.org/jira/browse/SOLR-356
>             Project: Solr
>          Issue Type: New Feature
>            Reporter: Yonik Seeley
>         Attachments: pluggableFunctions.patch
>
>
> allow configuration of new value sources ot be created by the function query 
> parser.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to