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

Sean Owen commented on MAHOUT-644:
----------------------------------

That's my "fault", I changed that behavior. In the case that mapper.getVector() 
was null, the iterator was returning a null. Is that normal behavior? It 
doesn't sound right. It means that TermVectorMapper.setExpectations() was never 
called.

Either...
- That's a problem and this has uncovered it, so, what's your input that 
triggers this?
- Or it's normal, and signals end of iteration
- Or it's normal, and we really should return a null in the iteration

Do you know what's the right thing?

> LuceneIterable does not throw exception if field does not have term vectors
> ---------------------------------------------------------------------------
>
>                 Key: MAHOUT-644
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-644
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Utils
>    Affects Versions: 0.4
>            Reporter: Frank Scholten
>            Assignee: Sean Owen
>            Priority: Minor
>             Fix For: 0.5
>
>         Attachments: MAHOUT-644.patch, MAHOUT-644.patch
>
>
> LuceneIterable does not throw an exception when it tries to create Mahout 
> vectors from a field without term vectors. It should throw an 
> IllegalStateException and not silently continue and return null vectors.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to