Skip to content

Conversation

@skambha
Copy link
Contributor

@skambha skambha commented Feb 14, 2017

What changes were proposed in this pull request?

https://spark.apache.org/docs/latest/sql-programming-guide.html#caching-data-in-memory
In the doc, the call spark.cacheTable(“tableName”) and spark.uncacheTable(“tableName”) actually needs to be spark.catalog.cacheTable and spark.catalog.uncacheTable

How was this patch tested?

Built the docs and verified the change shows up fine.

@gatorsmile
Copy link
Member

test this please

@SparkQA
Copy link

SparkQA commented Feb 14, 2017

Test build #72854 has finished for PR 16919 at commit cab668e.

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

@gatorsmile
Copy link
Member

gatorsmile commented Feb 14, 2017

LGTM. Merging to master/2.1/2.0!

asfgit pushed a commit that referenced this pull request Feb 14, 2017
…in the doc

## What changes were proposed in this pull request?

https://spark.apache.org/docs/latest/sql-programming-guide.html#caching-data-in-memory
In the doc, the call spark.cacheTable(“tableName”) and spark.uncacheTable(“tableName”) actually needs to be spark.catalog.cacheTable and spark.catalog.uncacheTable

## How was this patch tested?
Built the docs and verified the change shows up fine.

Author: Sunitha Kambhampati <skambha@us.ibm.com>

Closes #16919 from skambha/docChange.

(cherry picked from commit 9b5e460)
Signed-off-by: Xiao Li <gatorsmile@gmail.com>
@asfgit asfgit closed this in 9b5e460 Feb 14, 2017
@skambha
Copy link
Contributor Author

skambha commented Feb 14, 2017

Thanks Xiao!

@skambha skambha deleted the docChange branch February 14, 2017 17:51
cmonkey pushed a commit to cmonkey/spark that referenced this pull request Feb 15, 2017
…in the doc

## What changes were proposed in this pull request?

https://spark.apache.org/docs/latest/sql-programming-guide.html#caching-data-in-memory
In the doc, the call spark.cacheTable(“tableName”) and spark.uncacheTable(“tableName”) actually needs to be spark.catalog.cacheTable and spark.catalog.uncacheTable

## How was this patch tested?
Built the docs and verified the change shows up fine.

Author: Sunitha Kambhampati <skambha@us.ibm.com>

Closes apache#16919 from skambha/docChange.
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.

3 participants