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

Hoss Man updated SOLR-4893:
---------------------------

    Attachment: SOLR-4893.patch

updated patch that addresses a few of my previous comments...

{quote}
* would be good to have a test of fieldNameMatchesSchemaField in an excludes
* i don't think your ignore-not-in-schema-and-foo-name-prefix is verifying that 
the foo prefix is required?
* even with your added comments, the logic involving 
fieldNameMatchesSchemaField in shouldMutate seems kind of hairy (too hairy for 
me to be confident it's correct by reading it w/o looking at the test) .. i'm 
wondering if it can't be "unwound" a bit to make it more straight forward?
{quote}

...lemme know what you think of the new logic in 
ConfigurableFieldNameSelector.shouldMutate .. i think it's easier to read, but 
i'm curious if that's just how my mind works

I retract this suggestion...

bq. you should change getDefaultSelector in IgnoreFieldUpdateProcessorFactory 
to take advantage of your new logic (ie: call createFieldNameSelector(...))

..no reason to make that method reuse createFieldNameSelector with 
anartificials set of params just to get a selector that would be slower then 
what it uses right now.
                
> Add a FieldMutatingUpdateProcessor FieldNameSelector that checks whether a 
> field matches any schema field 
> ----------------------------------------------------------------------------------------------------------
>
>                 Key: SOLR-4893
>                 URL: https://issues.apache.org/jira/browse/SOLR-4893
>             Project: Solr
>          Issue Type: Improvement
>          Components: update
>            Reporter: Steve Rowe
>            Assignee: Steve Rowe
>            Priority: Minor
>         Attachments: SOLR-4893.patch, SOLR-4893.patch
>
>
> Add a new field update processor selector that will configure the processor 
> to select fields that match any schema field, or that don't match any schema 
> field, depending on its boolean parameter, e.g. to select fields that don't 
> match any schema field:
> {code:xml}
> <bool name="fieldNameMatchesSchemaField">false</bool>
> {code}

--
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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to