Tony Foerster has posted comments on this change. ( http://gerrit.cloudera.org:8080/11030 )
Change subject: Scala code formatting with Scalafmt ...................................................................... Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/11030/6/java/.gitignore File java/.gitignore: http://gerrit.cloudera.org:8080/#/c/11030/6/java/.gitignore@39 PS6, Line 39: **/out let me know if you want this as part of a different commit. http://gerrit.cloudera.org:8080/#/c/11030/3/java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/SparkUtil.scala File java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/SparkUtil.scala: http://gerrit.cloudera.org:8080/#/c/11030/3/java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/SparkUtil.scala@25 PS3, Line 25: case Type.BOOL => BooleanType > You may need to undo the changes and then re-run the formatting. My underst Check out patch set 6, it's fixed there. The formatter will un-align, this patch set was not reading .scalafmt.conf and was instead using the default. -- To view, visit http://gerrit.cloudera.org:8080/11030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iac96383d88394084e19712177d05f9fc63de766c Gerrit-Change-Number: 11030 Gerrit-PatchSet: 6 Gerrit-Owner: Tony Foerster <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tony Foerster <[email protected]> Gerrit-Comment-Date: Tue, 24 Jul 2018 22:05:43 +0000 Gerrit-HasComments: Yes
