Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20070#discussion_r159143849
  
    --- Diff: 
examples/src/main/scala/org/apache/spark/examples/mllib/LatentDirichletAllocationExample.scala
 ---
    @@ -46,7 +46,10 @@ object LatentDirichletAllocationExample {
         val topics = ldaModel.topicsMatrix
         for (topic <- Range(0, 3)) {
           print(s"Topic $topic :")
    -      for (word <- Range(0, ldaModel.vocabSize)) { print(s" ${topics(word, 
topic)}") }
    +      for (word <- Range(0, ldaModel.vocabSize))
    +          {
    --- End diff --
    
    No, we put the open brace on the preceding line with for, and don't triple 
indent. See any other for loop in the code.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to