Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12208 )
Change subject: [examples] a small update on SparkExample ...................................................................... Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/12208/2/examples/scala/spark-example/src/main/scala/org/apache/kudu/spark/examples/SparkExample.scala File examples/scala/spark-example/src/main/scala/org/apache/kudu/spark/examples/SparkExample.scala: http://gerrit.cloudera.org:8080/#/c/12208/2/examples/scala/spark-example/src/main/scala/org/apache/kudu/spark/examples/SparkExample.scala@44 PS2, Line 44: val tableReplicasNum: Int = Integer.getInteger("tableReplicasNum", 1) Since this is an example that a user with no experience may use, can you add a comment about a replication factor of 1 being a bad idea. I understand it's set here for maximum environment compatibility. http://gerrit.cloudera.org:8080/#/c/12208/2/examples/scala/spark-example/src/main/scala/org/apache/kudu/spark/examples/SparkExample.scala@81 PS2, Line 81: if (!kuduContext.tableExists(tableName)) { Instead should we just fail if it already exists? -- To view, visit http://gerrit.cloudera.org:8080/12208 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icf8c4e675ca6c240582242658fa8173a1ccd271d Gerrit-Change-Number: 12208 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 10 Jan 2019 03:55:56 +0000 Gerrit-HasComments: Yes
