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

Hoss Man commented on SOLR-4892:
--------------------------------

+1 ... code looks good.

two minor things i didn't notice before...

a) i think there's some class javadoc cut/paste mistakes ... if you grep the 
patch for "solr.ParseDoubleFieldUpdateProcessorFactory" it shows up in the 
class jdocs for several other factories.

b) the logic in ParseBooleanFieldUpdateProcessorFactory could probably be a 
little simpler/faster if there was a single {{Map<String,Boolean>}} instead of 
two {{Set<String>}} that each had to be checked.
                
> Add field update processors to parse/convert String-typed fields to Date, 
> Number, and Boolean
> ---------------------------------------------------------------------------------------------
>
>                 Key: SOLR-4892
>                 URL: https://issues.apache.org/jira/browse/SOLR-4892
>             Project: Solr
>          Issue Type: New Feature
>          Components: update
>            Reporter: Steve Rowe
>            Assignee: Steve Rowe
>            Priority: Minor
>             Fix For: 4.4
>
>         Attachments: SOLR-4892.patch, SOLR-4892.patch
>
>
> Add {{FieldMutatingUpdateProcessorFactory}} subclasses 
> {{ParseFooUpdateProcessorFactory}}, where {{Foo}} includes {{Date}}, 
> {{Double}}, {{Long}}, and {{Boolean}}.
> These factories will have a default selector that matches all fields that 
> either don’t match any schema field, or are in the schema with the 
> corresponding {{typeClass}}.  They can optionally support a list of multiple 
> format specifiers.  If they see a value that is not a CharSequence, or can't 
> parse the value using a configured format, they ignore it and leave it as is.
> For multi-valued fields, these processors will not convert any values unless 
> all are first successfully parsed.  Ordering the processors [Boolean, Long, 
> Double, Date] will allow e.g. values [2, 5, 8.6] to be left alone by the 
> Boolean and Long processors, but then converted by the Double processor.

--
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: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to