[ 
https://issues.apache.org/jira/browse/SOLR-8113?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Hoss Man updated SOLR-8113:
---------------------------
    Attachment: SOLR-8113.patch


Patch Updates...
* Fixed javadocs to refer to replaceAll (code change was already previous patch)
* added testCloneFieldRegexReplaceAll to prove replaceAll usecases would work.
** test initially failed due to the Matcher.replaceAll being conditional on a 
succesful "Mather.match()" call.
** Replaced "Mather.match()" with a "Matcher.find()" call to ensure replaceAll 
would work as intended
* Updated the javadocs example config (and corrisponding test config) refering 
to "ending in {{_price}}" to actually use string ending bounds.
* Updated the test configs to include the start/end boundary rules for 
({{^feat(.*)s$}}) so it matches the javadocs


...i plan to commit & start backporting as soon as my full test + precommit 
finishes

> Accept replacement strings in CloneFieldUpdateProcessorFactory
> --------------------------------------------------------------
>
>                 Key: SOLR-8113
>                 URL: https://issues.apache.org/jira/browse/SOLR-8113
>             Project: Solr
>          Issue Type: Improvement
>          Components: update
>    Affects Versions: 5.3
>            Reporter: Gus Heck
>            Assignee: Hoss Man
>         Attachments: SOLR-8113.patch, SOLR-8113.patch, SOLR-8113.patch, 
> SOLR-8113.patch, SOLR-8113.patch
>
>
> Presently CloneFieldUpdateProcessorFactory accepts regular expressions to 
> select source fields, which mirrors wildcards in the source for copyField in 
> the schema. This patch adds a counterpart to copyField's wildcards in the 
> dest attribute by interpreting the dest parameter as a regex replacement 
> string.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to