Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-30812] [SQL] update the skew join configs by adding the prefix "skewedJoinOptimization" #27630

Closed
wants to merge 1 commit into from

Conversation

JkSelf
Copy link
Contributor

@JkSelf JkSelf commented Feb 19, 2020

What changes were proposed in this pull request?

This is a follow up in PR#27563.
This PR adds the prefix of "skewedJoinOptimization" in the skew join related configs.

Why are the changes needed?

address remaining address

Does this PR introduce any user-facing change?

No

How was this patch tested?

only update config and no need new ut.

@SparkQA
Copy link

SparkQA commented Feb 19, 2020

Test build #118660 has finished for PR 27630 at commit 2734b24.

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

@cloud-fan cloud-fan closed this in 10a4eaf Feb 19, 2020
@cloud-fan
Copy link
Contributor

thanks, merging to master/3.0!

cloud-fan pushed a commit that referenced this pull request Feb 19, 2020
…"skewedJoinOptimization"

### What changes were proposed in this pull request?
This is a follow up in [PR#27563](#27563).
This PR adds the prefix of "skewedJoinOptimization" in the skew join related configs.

### Why are the changes needed?
address remaining address

### Does this PR introduce any user-facing change?
No

### How was this patch tested?
only update config and no need new ut.

Closes #27630 from JkSelf/renameskewjoinconfig.

Authored-by: jiake <ke.a.jia@intel.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
(cherry picked from commit 10a4eaf)
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
sjincho pushed a commit to sjincho/spark that referenced this pull request Apr 15, 2020
…"skewedJoinOptimization"

### What changes were proposed in this pull request?
This is a follow up in [PR#27563](apache#27563).
This PR adds the prefix of "skewedJoinOptimization" in the skew join related configs.

### Why are the changes needed?
address remaining address

### Does this PR introduce any user-facing change?
No

### How was this patch tested?
only update config and no need new ut.

Closes apache#27630 from JkSelf/renameskewjoinconfig.

Authored-by: jiake <ke.a.jia@intel.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants