Skip to content

Conversation

@lianhuiwang
Copy link
Contributor

@lianhuiwang lianhuiwang commented May 10, 2016

What changes were proposed in this pull request?

  1. add query fragment for execute stage according to exchange.
  2. before running no leaf query fragment, optimize plan using mapOutputStatistics of its child fragments.
    if size of one exchange is less than autoBroadcastJoinThreshold, it transforms sort merge join to broadcast join.
    I will support optimization for skew join in another PR.

How was this patch tested?

unit tests

@SparkQA
Copy link

SparkQA commented May 10, 2016

Test build #58239 has finished for PR 13028 at commit 73de7b2.

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

@SparkQA
Copy link

SparkQA commented May 10, 2016

Test build #58240 has finished for PR 13028 at commit 6cc187e.

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

@SparkQA
Copy link

SparkQA commented May 10, 2016

Test build #58241 has finished for PR 13028 at commit 6f1105c.

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

@SparkQA
Copy link

SparkQA commented May 11, 2016

Test build #58376 has finished for PR 13028 at commit 5de46b0.

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

@lianhuiwang lianhuiwang changed the title [SPARK-9860][SQL][WIP] Join: Determine the join strategy (broadcast join or shuffle join) at runtime [SPARK-9860][SQL] Join: Determine the join strategy (broadcast join or shuffle join) at runtime May 11, 2016
@SparkQA
Copy link

SparkQA commented May 11, 2016

Test build #58378 has finished for PR 13028 at commit a18be55.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class QueryFragmentSuite extends QueryTest with SQLTestUtils with SharedSQLContext

@lianhuiwang
Copy link
Contributor Author

cc @rxin @davies

@SparkQA
Copy link

SparkQA commented May 12, 2016

Test build #58481 has finished for PR 13028 at commit d9c66d7.

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

@sameeragarwal
Copy link
Member

@lianhuiwang could this regress performance for existing queries? Can you please share some benchmarks with us (if any)?

@gatorsmile
Copy link
Member

We are closing it due to inactivity. please do reopen if you want to push it forward. Thanks!

@asfgit asfgit closed this in b32bd00 Jun 27, 2017
@meteorchenwu
Copy link

It is great for BI scene. Why not continue?

zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 22, 2025
## What changes were proposed in this pull request?

This PR proposes to close stale PRs, mostly the same instances with apache#18017

I believe the author in apache#14807 removed his account.

Closes apache#7075
Closes apache#8927
Closes apache#9202
Closes apache#9366
Closes apache#10861
Closes apache#11420
Closes apache#12356
Closes apache#13028
Closes apache#13506
Closes apache#14191
Closes apache#14198
Closes apache#14330
Closes apache#14807
Closes apache#15839
Closes apache#16225
Closes apache#16685
Closes apache#16692
Closes apache#16995
Closes apache#17181
Closes apache#17211
Closes apache#17235
Closes apache#17237
Closes apache#17248
Closes apache#17341
Closes apache#17708
Closes apache#17716
Closes apache#17721
Closes apache#17937

Added:
Closes apache#14739
Closes apache#17139
Closes apache#17445
Closes apache#18042
Closes apache#18359

Added:
Closes apache#16450
Closes apache#16525
Closes apache#17738

Added:
Closes apache#16458
Closes apache#16508
Closes apache#17714

Added:
Closes apache#17830
Closes apache#14742

## How was this patch tested?

N/A

Author: hyukjinkwon <[email protected]>

Closes apache#18417 from HyukjinKwon/close-stale-pr.
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.

5 participants