Skip to content

Conversation

@liancheng
Copy link
Contributor

@liancheng liancheng commented Jul 18, 2016

What changes were proposed in this pull request?

This PR moves one and the last hard-coded Scala example snippet from the SQL programming guide into SparkSqlExample.scala. It also renames all Scala/Java example files so that all "Sql" in the file names are updated to "SQL".

How was this patch tested?

Manually verified the generated HTML page.

@yhuai
Copy link
Contributor

yhuai commented Jul 18, 2016

LGTM. Can we reuse a existing jira number?

@SparkQA
Copy link

SparkQA commented Jul 18, 2016

Test build #62449 has finished for PR 14245 at commit e797c30.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@rxin
Copy link
Contributor

rxin commented Jul 18, 2016

FWIW I think it'd be better to name the file SparkSQLExamples, rather than SparkSqlExamples. It just feels weird to have SparkSql. And I'm talking about both Scala and Python.

@liancheng liancheng changed the title [MINOR][DOCS][EXAMPLES] Minor Scala example update [SPARK-16303][DOCS][EXAMPLES] Minor Scala example update Jul 18, 2016
@liancheng liancheng changed the title [SPARK-16303][DOCS][EXAMPLES] Minor Scala example update [SPARK-16303][DOCS][EXAMPLES] Minor Scala/Jva example update Jul 18, 2016
@liancheng liancheng changed the title [SPARK-16303][DOCS][EXAMPLES] Minor Scala/Jva example update [SPARK-16303][DOCS][EXAMPLES] Minor Scala/Java example update Jul 18, 2016
@liancheng
Copy link
Contributor Author

Reused JIRA number SPARK-16303 and renamed Scala/Java example file names. Python examples are not being updated to use the include_example tag yet. The PR (#14098) is still in WIP status.

@liancheng liancheng force-pushed the minor-scala-example-update branch from 03491fd to 927c46a Compare July 18, 2016 09:49
@SparkQA
Copy link

SparkQA commented Jul 18, 2016

Test build #62462 has finished for PR 14245 at commit 927c46a.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jul 18, 2016

Test build #62461 has finished for PR 14245 at commit 03491fd.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@yhuai
Copy link
Contributor

yhuai commented Jul 19, 2016

Thanks. Merging to master and branch 2.0.

asfgit pushed a commit that referenced this pull request Jul 19, 2016
## What changes were proposed in this pull request?

This PR moves one and the last hard-coded Scala example snippet from the SQL programming guide into `SparkSqlExample.scala`. It also renames all Scala/Java example files so that all "Sql" in the file names are updated to "SQL".

## How was this patch tested?

Manually verified the generated HTML page.

Author: Cheng Lian <lian@databricks.com>

Closes #14245 from liancheng/minor-scala-example-update.

(cherry picked from commit 1426a08)
Signed-off-by: Yin Huai <yhuai@databricks.com>
@asfgit asfgit closed this in 1426a08 Jul 19, 2016
@liancheng liancheng deleted the minor-scala-example-update branch July 20, 2016 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants