[
https://issues.apache.org/jira/browse/BLUR-44?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13530611#comment-13530611
]
Aaron McCurry commented on BLUR-44:
-----------------------------------
I think that we should focus on just the none default analyzers at this point
and we should think more about the FieldManager. I have tried implementing my
own IndexableField object today and we need to think through that more.
Overall the code in the patch looks good except for catching the reflection
exceptions and not throwing them as a runtime or io exception, or maybe blur
exception. What do you think?
> Update BlurAnalyzer to use the Blur Thrift Analyzer object.
> -----------------------------------------------------------
>
> Key: BLUR-44
> URL: https://issues.apache.org/jira/browse/BLUR-44
> Project: Apache Blur
> Issue Type: Improvement
> Affects Versions: 0.2.0
> Environment: 0.2-dev-removing-old-thrift branch
> Reporter: Aaron McCurry
> Fix For: 0.2.0
>
> Attachments: 0001-BLUR-ID-44-created-Blur-Analyzer.patch,
> BlurAnalyzer.java
>
>
> The BlurAnalyzer constructor that takes a
> org.apache.blur.thrift.generated.Analyzer currently does nothing. Obviously
> this needs to be implemented, I have added javadocs to the Blur thrift API
> that describes the expected behavior.
--
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
