Skip to content

Conversation

@ueshin
Copy link
Member

@ueshin ueshin commented Sep 26, 2018

What changes were proposed in this pull request?

In Scala, HiveContext sets a config spark.sql.catalogImplementation of the given SparkContext and then passes to SparkSession.builder.
The HiveContext in PySpark should behave as the same as Scala.

How was this patch tested?

Existing tests.

@ueshin
Copy link
Member Author

ueshin commented Sep 26, 2018

cc @cloud-fan @gatorsmile

@SparkQA
Copy link

SparkQA commented Sep 26, 2018

Test build #96604 has finished for PR 22552 at commit 858941b.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@ueshin
Copy link
Member Author

ueshin commented Sep 26, 2018

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented Sep 26, 2018

Test build #96610 has finished for PR 22552 at commit 858941b.

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

@cloud-fan
Copy link
Contributor

thanks, merging to master/2.4!

asfgit pushed a commit that referenced this pull request Sep 27, 2018
… same as Scala.

## What changes were proposed in this pull request?

In Scala, `HiveContext` sets a config `spark.sql.catalogImplementation` of the given `SparkContext` and then passes to `SparkSession.builder`.
The `HiveContext` in PySpark should behave as the same as Scala.

## How was this patch tested?

Existing tests.

Closes #22552 from ueshin/issues/SPARK-25540/hive_context.

Authored-by: Takuya UESHIN <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
(cherry picked from commit c3c45cb)
Signed-off-by: Wenchen Fan <[email protected]>
@asfgit asfgit closed this in c3c45cb Sep 27, 2018
Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

LGTM2

@HyukjinKwon
Copy link
Member

hm, why did we merge this into 2.4 btw?

@cloud-fan
Copy link
Contributor

This is part of the bug fix PR: #22545 . I asked @ueshin to separate it since this one is self-contained.

daspalrahul pushed a commit to daspalrahul/spark that referenced this pull request Sep 29, 2018
… same as Scala.

## What changes were proposed in this pull request?

In Scala, `HiveContext` sets a config `spark.sql.catalogImplementation` of the given `SparkContext` and then passes to `SparkSession.builder`.
The `HiveContext` in PySpark should behave as the same as Scala.

## How was this patch tested?

Existing tests.

Closes apache#22552 from ueshin/issues/SPARK-25540/hive_context.

Authored-by: Takuya UESHIN <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
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