[
https://issues.apache.org/jira/browse/SOLR-4893?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Steve Rowe updated SOLR-4893:
-----------------------------
Attachment: SOLR-4893.patch
{quote}
updated patch that addresses a few of my previous comments...
bq. * would be good to have a test of fieldNameMatchesSchemaField in an excludes
bq. * i don't think your ignore-not-in-schema-and-foo-name-prefix is verifying
that the foo prefix is required?
{quote}
Thanks, these look good.
{quote}
bq. 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?
...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
{quote}
It looks easier to read to me, thanks.
{quote}
* i would make getBooleanArg give an explict error if the key exists multiple
times
* to be nice to users who might not no any better, i would suggest that
getBooleanArg should also accept a String and pass it to Boolean.valueOf
{quote}
The attached patch implements the above two suggestions. I think that covers
your comments?
> 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, 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]
