Skip to content

Conversation

@gengliangwang
Copy link
Member

@gengliangwang gengliangwang commented Oct 5, 2018

What changes were proposed in this pull request?

By replacing loops with random possible value.

  • read partitioning bucketed tables with bucket pruning filters reduce from 55s to 7s
  • read partitioning bucketed tables having composite filters reduce from 54s to 8s
  • total time: reduce from 288s to 192s

How was this patch tested?

Unit test

.bucketBy(8, "j", "k")
.saveAsTable("bucketed_table")

for (i <- 0 until 5) {
Copy link
Member

@HyukjinKwon HyukjinKwon Oct 5, 2018

Choose a reason for hiding this comment

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

Oh haha I was struggling to find why it was 5 and 13 and looks it's in df :-)

@HyukjinKwon
Copy link
Member

cc @cloud-fan too for sure.

@SparkQA
Copy link

SparkQA commented Oct 5, 2018

Test build #96988 has finished for PR 22640 at commit 58c419a.

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

@SparkQA
Copy link

SparkQA commented Oct 5, 2018

Test build #97004 has finished for PR 22640 at commit 1c5bfef.

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

@HyukjinKwon
Copy link
Member

Merged to master.

@asfgit asfgit closed this in 1ee472e Oct 6, 2018
jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
…HiveSupportSuite

## What changes were proposed in this pull request?

By replacing loops with random possible value.
- `read partitioning bucketed tables with bucket pruning filters` reduce from 55s to 7s
- `read partitioning bucketed tables having composite filters` reduce from 54s to 8s
- total time: reduce from 288s to 192s

## How was this patch tested?

Unit test

Closes apache#22640 from gengliangwang/fastenBucketedReadSuite.

Authored-by: Gengliang Wang <[email protected]>
Signed-off-by: hyukjinkwon <[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.

3 participants