[kudu-CR] [docs] fix spark integration examples

["Grant Henke (Code Review)" <ge...@cloudera.org>]

Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13927 )

Change subject: [docs] fix spark integration examples
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/13927/1/docs/developing.adoc
File docs/developing.adoc:

http://gerrit.cloudera.org:8080/#/c/13927/1/docs/developing.adoc@112
PS1, Line 112: Below is a minimal Spark SQL "select" example. We first import the kudu spark package,
You don't need to do this, but this could be a good opportunity to link to the Spark quickstart: 
https://github.com/apache/kudu/tree/master/examples/quickstart/spark


http://gerrit.cloudera.org:8080/#/c/13927/1/docs/developing.adoc@116
PS1, Line 116: NOTE: You can use Kudu CLI tool to create table and generate data by
Unfortunately this table will haven a random name instead of `default.my_table`. Can we instead show Spark code that creates the table?


http://gerrit.cloudera.org:8080/#/c/13927/1/docs/developing.adoc@124
PS1, Line 124: val df = spark.read.options(Map("kudu.master" -> "kudu.master:7051",
Why reduce this to a single master? Given multi-maser is a more common production scenario, I think it's good to show 3.



-- 
To view, visit http://gerrit.cloudera.org:8080/13927
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I2a48cc34aea3cc42afd48f43e142a669081f14e2
Gerrit-Change-Number: 13927
Gerrit-PatchSet: 1
Gerrit-Owner: Yingchun Lai <40...@qq.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <40...@qq.com>
Gerrit-Comment-Date: Mon, 12 Aug 2019 17:58:55 +0000
Gerrit-HasComments: Yes