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

David Eric Pugh commented on TIKA-2368:
---------------------------------------

I'm actually not sure I touched {{SentimentParser}}, as everything  I did was 
in the {{tika-nlp}} project, which appears to only have a {{AgePredictor}} 
client.   That client is what drags in all the large list of dependencies.   I 
ended up commenting on this ticket because of the comment in {{tika-nlp}} that 
said "fix me when TIKA-2368" is fixed.  

You are right though, that all the changes I made, well, we just keep the flag 
to say "don't alert on failure" ;)

> Clean up SentimentParser dependencies
> -------------------------------------
>
>                 Key: TIKA-2368
>                 URL: https://issues.apache.org/jira/browse/TIKA-2368
>             Project: Tika
>          Issue Type: Improvement
>            Reporter: Tim Allison
>            Priority: Blocker
>
> Is there any way to avoid reliance on edu.usc.ir's sentiment-analysis-parser? 
>  I ask because:
> {noformat}
> [WARNING] sentiment-analysis-parser-0.1.jar, tika-parsers-1.15-SNAPSHOT.jar 
> define 1 overlapping classes: 
> [WARNING]   - org.apache.tika.parser.sentiment.analysis.SentimentParser
> [WARNING] tika-core-1.15-SNAPSHOT.jar, tika-translate-1.15-SNAPSHOT.jar 
> define 4 overlapping classes: 
> [WARNING]   - org.apache.tika.language.translate.DefaultTranslator$1
> [WARNING]   - org.apache.tika.language.translate.EmptyTranslator
> [WARNING]   - org.apache.tika.language.translate.DefaultTranslator
> [WARNING]   - org.apache.tika.language.translate.Translator
> {noformat}
> We should be ok keeping things as they are and excluding SentimentParser and 
> tika-translate, but can we easily move the code that's still in edu.usc.ir's 
> package into Tika?



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to