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

jirapos...@reviews.apache.org commented on MAHOUT-987:
------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4238/
-----------------------------------------------------------

Review request for mahout.


Summary
-------

Generated with:

find . -name *java | xargs perl -pi -e 's/(if|while|for)(/$1 (/'


This addresses bug MAHOUT-987.
    https://issues.apache.org/jira/browse/MAHOUT-987


Diffs
-----

  
trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/recommender/knn/ConjugateGradientOptimizer.java
 1298262 
  trunk/core/src/main/java/org/apache/mahout/classifier/ConfusionMatrix.java 
1298262 
  trunk/core/src/main/java/org/apache/mahout/clustering/lda/LDADriver.java 
1298262 
  trunk/core/src/main/java/org/apache/mahout/clustering/lda/LDASampler.java 
1298262 
  
trunk/core/src/main/java/org/apache/mahout/clustering/lda/cvb/CVB0DocInferenceMapper.java
 1298262 
  trunk/core/src/main/java/org/apache/mahout/clustering/lda/cvb/CVB0Driver.java 
1298262 
  
trunk/core/src/main/java/org/apache/mahout/clustering/lda/cvb/CachingCVB0Mapper.java
 1298262 
  
trunk/core/src/main/java/org/apache/mahout/clustering/lda/cvb/CachingCVB0PerplexityMapper.java
 1298262 
  
trunk/core/src/main/java/org/apache/mahout/clustering/lda/cvb/InMemoryCollapsedVariationalBayes0.java
 1298262 
  
trunk/core/src/main/java/org/apache/mahout/clustering/lda/cvb/ModelTrainer.java 
1298262 
  trunk/core/src/main/java/org/apache/mahout/clustering/lda/cvb/TopicModel.java 
1298262 
  
trunk/core/src/main/java/org/apache/mahout/clustering/spectral/eigencuts/EigencutsDriver.java
 1298262 
  
trunk/core/src/main/java/org/apache/mahout/common/distance/MahalanobisDistanceMeasure.java
 1298262 
  
trunk/core/src/main/java/org/apache/mahout/math/DistributedRowMatrixWriter.java 
1298262 
  trunk/core/src/main/java/org/apache/mahout/math/MatrixUtils.java 1298262 
  
trunk/core/src/main/java/org/apache/mahout/math/hadoop/decomposer/DistributedLanczosSolver.java
 1298262 
  
trunk/core/src/main/java/org/apache/mahout/math/hadoop/decomposer/HdfsBackedLanczosState.java
 1298262 
  trunk/core/src/main/java/org/apache/mahout/math/stats/Sampler.java 1298262 
  
trunk/core/src/test/java/org/apache/mahout/clustering/ClusteringTestUtils.java 
1298262 
  trunk/core/src/test/java/org/apache/mahout/clustering/lda/TestMapReduce.java 
1298262 
  
trunk/core/src/test/java/org/apache/mahout/clustering/lda/cvb/TestCVBModelTrainer.java
 1298262 
  
trunk/core/src/test/java/org/apache/mahout/common/distance/DefaultDistanceMeasureTest.java
 1298262 
  trunk/core/src/test/java/org/apache/mahout/math/MatrixWritableTest.java 
1298262 
  
trunk/core/src/test/java/org/apache/mahout/math/hadoop/TestDistributedRowMatrix.java
 1298262 
  
trunk/core/src/test/java/org/apache/mahout/math/hadoop/decomposer/TestDistributedLanczosSolver.java
 1298262 
  
trunk/core/src/test/java/org/apache/mahout/math/hadoop/decomposer/TestDistributedLanczosSolverCLI.java
 1298262 
  
trunk/integration/src/main/java/org/apache/mahout/clustering/lda/LDAPrintTopics.java
 1298262 
  trunk/integration/src/main/java/org/apache/mahout/utils/MatrixDumper.java 
1298262 
  
trunk/integration/src/main/java/org/apache/mahout/utils/vectors/VectorHelper.java
 1298262 
  
trunk/math/src/main/java/org/apache/mahout/math/decomposer/lanczos/LanczosSolver.java
 1298262 
  trunk/math/src/main/java/org/apache/mahout/math/jet/stat/Gamma.java 1298262 
  
trunk/math/src/main/java/org/apache/mahout/math/matrix/linalg/EigenvalueDecomposition.java
 1298262 
  trunk/math/src/main/java/org/apache/mahout/math/matrix/linalg/Property.java 
1298262 
  trunk/math/src/test/java/org/apache/mahout/math/MatrixTest.java 1298262 
  
trunk/math/src/test/java/org/apache/mahout/math/TestSingularValueDecomposition.java
 1298262 
  trunk/math/src/test/java/org/apache/mahout/math/decomposer/SolverTest.java 
1298262 
  
trunk/math/src/test/java/org/apache/mahout/math/decomposer/hebbian/TestHebbianSolver.java
 1298262 
  
trunk/math/src/test/java/org/apache/mahout/math/decomposer/lanczos/TestLanczosSolver.java
 1298262 

Diff: https://reviews.apache.org/r/4238/diff


Testing
-------


Thanks,

tom


                
> Our build is unstable - this should reduce our style warnings by >200 
> ----------------------------------------------------------------------
>
>                 Key: MAHOUT-987
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-987
>             Project: Mahout
>          Issue Type: Improvement
>    Affects Versions: 0.7
>            Reporter: tom pierce
>             Fix For: 0.7
>
>         Attachments: MAHOUT-987.patch
>
>
> If we're going to keep these Jenkins style rules, let's get our build stable!
> Here's about 200 small fixes created by:
> find . -name \*java | xargs perl -pi -e 's/(if|while|for)\(/$1 (/'
> Any objections?

--
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