Skip to content

Conversation

@clockfly
Copy link
Contributor

@clockfly clockfly commented Jun 13, 2016

What changes were proposed in this pull request?

Revert partial changes in SPARK-12600, and add some deprecated method back to SQLContext for backward source code compatibility.

How was this patch tested?

Manual test.

@SparkQA
Copy link

SparkQA commented Jun 13, 2016

Test build #60387 has finished for PR 13637 at commit fd08f1d.

  • This patch fails MiMa tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@clockfly clockfly changed the title [SPARK-15914][SQL] Add deprecated method back to SQLContext for backward compatibility [SPARK-15914][SQL] Add deprecated method back to SQLContext for backward source code compatibility Jun 13, 2016
@SparkQA
Copy link

SparkQA commented Jun 13, 2016

Test build #60424 has finished for PR 13637 at commit 4c039d8.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jun 13, 2016

Test build #60425 has finished for PR 13637 at commit fbbc031.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

* `read().format(source).schema(schema).options(options).load()`.
*/
@deprecated("Use read.format(source).schema(schema).options(options).load() instead.", "1.4.0")
def load(source: String, schema: StructType, options: java.util.Map[String, String]): DataFrame =
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit the right style is:

def load(
    sources: String,
    schema: StructType,
    options: ...): DataFrame = {
  read.format...
}

@andrewor14
Copy link
Contributor

This looks good, just minor commnts

@SparkQA
Copy link

SparkQA commented Jun 13, 2016

Test build #60433 has finished for PR 13637 at commit 04ef1b5.

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

@SparkQA
Copy link

SparkQA commented Jun 14, 2016

Test build #60447 has finished for PR 13637 at commit 0cc81b1.

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

@yhuai
Copy link
Contributor

yhuai commented Jun 14, 2016

Thanks. Merging to master and branch 2.0.

asfgit pushed a commit that referenced this pull request Jun 14, 2016
…ard source code compatibility

## What changes were proposed in this pull request?

Revert partial changes in SPARK-12600, and add some deprecated method back to SQLContext for backward source code compatibility.

## How was this patch tested?

Manual test.

Author: Sean Zhong <[email protected]>

Closes #13637 from clockfly/SPARK-15914.

(cherry picked from commit 6e8cdef)
Signed-off-by: Yin Huai <[email protected]>
@asfgit asfgit closed this in 6e8cdef Jun 14, 2016
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