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

Hudson commented on MAHOUT-729:
-------------------------------

Integrated in Mahout-Quality #877 (See 
[https://builds.apache.org/job/Mahout-Quality/877/])
    MAHOUT-729 still more cleanup
MAHOUT-729 more cleanup

ssc : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1134468
Files : 
* /mahout/trunk/src/conf/driver.classes.props

ssc : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1134462
Files : 
* 
/mahout/trunk/integration/src/test/java/org/apache/mahout/text/SequenceFilesFromMailArchivesTest.java
* 
/mahout/trunk/examples/src/main/java/org/apache/mahout/classifier/bayes/SplitBayesInput.java
* 
/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/bayes/BayesAlgorithm.java
* 
/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/bayes/SequenceFileModelReader.java
* 
/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/cbayes/CBayesThetaNormalizerMapper.java
* 
/mahout/trunk/core/src/main/java/org/apache/mahout/common/mapreduce/VectorSumReducer.java
* 
/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/bayes/CBayesAlgorithm.java
* 
/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/common/BayesTfIdfMapper.java
* 
/mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/ARFFIterator.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/math/hadoop/MathHelper.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/common/mapreduce
* 
/mahout/trunk/examples/src/test/java/org/apache/mahout/classifier/bayes/SplitBayesInputTest.java
* 
/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/bayes/BayesClassifierDriver.java
* 
/mahout/trunk/examples/src/main/java/org/apache/mahout/cf/taste/example/grouplens/GroupLensDataModel.java
* 
/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/hadoop/als/eval/ParallelFactorizationEvaluator.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/common/IOUtils.java
* 
/mahout/trunk/examples/src/main/java/org/apache/mahout/classifier/sgd/SimpleCsvExamples.java
* 
/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/hadoop/als/eval/InMemoryFactorizationEvaluator.java
* 
/mahout/trunk/examples/src/main/java/org/apache/mahout/cf/taste/example/netflix/TransposeToByUser.java
* 
/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/bayes/mapreduce/bayes/BayesThetaNormalizerMapper.java
* 
/mahout/trunk/examples/src/main/java/org/apache/mahout/cf/taste/example/bookcrossing/BookCrossingDataModel.java
* 
/mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc/GenericJDBCDataModel.java
* 
/mahout/trunk/integration/src/test/java/org/apache/mahout/utils/eval/ParallelFactorizationEvaluatorTest.java


> Refactoring: Use Maps.newHashMap() and Lists.newArrayList() from Guava 
> globally
> -------------------------------------------------------------------------------
>
>                 Key: MAHOUT-729
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-729
>             Project: Mahout
>          Issue Type: Improvement
>    Affects Versions: 0.6
>            Reporter: Sebastian Schelter
>            Assignee: Sebastian Schelter
>         Attachments: MAHOUT-729.patch
>
>
> I put on my refactoring hat today and started to replace all calls to "new 
> HashMap" with Maps.newHashMap() as well as all calls to "new ArrayList" with 
> Lists.newArrayList() from Guava. This should make the code much more readable 
> as the generic type of a variable needs to be written only once per line. I 
> also refactored some minor things I encountered along the way.

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

Reply via email to