Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12208 )
Change subject: [examples] a small update on SparkExample ...................................................................... Patch Set 2: (5 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@43 PS2, Line 43: avaialble > typo Done 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 ad Done 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: tableReplicasNum: Int = Integer.getInteger("tableReplicasNum" > nit: maybe tableNumReplicas? Done 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? That's a good idea since the test doesn't do introspection of the existing table and requires some schema. Andrew and I talked about that as well, but for the first cut decided to go with the original logic. OK, let me update that. 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)) { > At first I leaned towards going ahead if the table exists, but on second th Indeed. -- 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: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mitch Barnett <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Thu, 10 Jan 2019 06:33:18 +0000 Gerrit-HasComments: Yes
