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

Grant Ingersoll commented on MAHOUT-768:
----------------------------------------

bq. I think that there may be some users of collections out there somewhere 
that we don't know about so I didn't want to delete it completely. We should be 
able to get the jenkins build to not care about it.

But it is still in Math, no?

bq. One good strategy would be to turn off the code coverage and quality phases 
temporarily. 

I've separated them out into two separate builds.  Clover is now MahoutClover 
and disabled b/c of the clover license.
                
> Duplicated DoubleFunction in mahout and mahout-collections (mahout.math 
> package).
> ---------------------------------------------------------------------------------
>
>                 Key: MAHOUT-768
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-768
>             Project: Mahout
>          Issue Type: Improvement
>          Components: collections, Math
>    Affects Versions: 0.5, collections-1.0
>            Reporter: Dawid Weiss
>            Assignee: Ted Dunning
>            Priority: Minor
>             Fix For: 0.7
>
>
> DoubleFunction is duplicated in Mahout Math and Collections. There are also 
> other things inside m.math.* package that are kept there to avoid circular 
> dependencies... Simply removing DoubleFunction from collections is not going 
> to work because it is needed for compilation (again, circular dependency 
> between collections and math). I see two solutions:
> 1) extract the common definitions inside math.function.* into a separate 
> module. This is a clean solution, but obviously scatters the code even 
> further.
> 2) create a compilation-time, optional dependency on mahout 0.5 in 
> collections, remove the entire mahout.math.* subpackage from collections and 
> live with this. I don't know how Maven handles circular dependencies of this 
> type:
> collections [trunk] -> [optional, required at build time] mahout.math 0.5
> mahout.math [trunk] -> collections [trunk]
> it seems tricky and error-prone, but should also work.
> Looking for other ideas of cleaning this mess up, of course.
> Dawid

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to