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

Christine Poerschke commented on SOLR-13965:
--------------------------------------------

GraphHandler and StreamHandler code sharing/duplication was mentioned both here 
and on the pull request.

SOLR-13965.01.patch factors out a static 
{{StreamHandler.addExpressiblePlugins}} method which GraphHandler could then 
use too.

(Note that this does _not_ fix the 
{{SolrConfig.classVsSolrPluginInfo.get(Expressible.class)}} suspected bug that 
[~mdrob] mentioned on the PR – {{Expressible.class}} vs. 
{{Expressible.class.getName()}} was the suspected type mismatch there, right?)

> Adding new functions to GraphHandler should be same as Streamhandler
> --------------------------------------------------------------------
>
>                 Key: SOLR-13965
>                 URL: https://issues.apache.org/jira/browse/SOLR-13965
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: streaming expressions
>    Affects Versions: 8.3
>            Reporter: David Eric Pugh
>            Priority: Minor
>         Attachments: SOLR-13965.01.patch
>
>          Time Spent: 1h 40m
>  Remaining Estimate: 0h
>
> Currently you add new functions to GraphHandler differently than you do in 
> StreamHandler.  We should have one way of extending the handlers that support 
> streaming expressions.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to