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

Jan Høydahl commented on SOLR-9565:
-----------------------------------

Great with added flexibility. Few comments:
* Remember that there is a difference between an URP class and an (named) URP 
instance. It must be possible to instantiate the same URP several times with 
different params (e.g. {{CloneFieldUpdateProcessorFactory}}).
* Instead of the magic -UpdateProcessorFactory suffix magic, can we perhaps let 
all URPs be [named 
SPIs|https://lucene.apache.org/core/6_6_0/core/org/apache/lucene/util/NamedSPILoader.NamedSPI.html]
 like tokenfilters and codecs, and use the short-name as key, e.g. 
{{processor=htmlstripfield}}?
* Why continue inventing dot-separated config options for complex stuff like 
this when we are moving towards JSON elsewhere? 
{{&HTMLStripField.fieldName=<some-field>&HTMLStripField.otherParam=<some-value>...}}
 could instead be {code}/solr/update?processor={"name": "htmlstripfield", 
"fieldName": "foo"}&processor={...}{code} or more complex example:
 {code}/solr/update?processor={"name": "clonefield", "source": {"fieldRegex": 
".*_price$", "exclude": ["list_price", "other"]}, "dest": "all_prices"}{code}

> Make every UpdateRequestProcessor available implicitly
> ------------------------------------------------------
>
>                 Key: SOLR-9565
>                 URL: https://issues.apache.org/jira/browse/SOLR-9565
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Noble Paul
>
> Now that we can 'construct' the URP chains through request parameters, we 
> should make all the URPs available automatically. The next challenge is to 
> make them read the configuration from request parameters as well
> to access {{HTMLStripFieldUpdateProcessorFactory}} the parameter could be 
> {{processor=HTMLStripField}} (The UpdateProcessorFactory part is 
> automatically appended )
> The next step is to make the URPs accept request parameters instead of just 
> configuration parameters e.g: 
> {{processor=HTMLStripField&HTMLStripField.fieldName=<some-field>}}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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

Reply via email to