Skip to content

Conversation

@wangyum
Copy link
Member

@wangyum wangyum commented Sep 20, 2018

What changes were proposed in this pull request?

Refactor UnsafeArrayDataBenchmark to use main method.
Generate benchmark result:

SPARK_GENERATE_BENCHMARK_FILES=1 build/sbt "sql/test:runMain org.apache.spark.sql.execution.benchmark.UnsafeArrayDataBenchmark"

How was this patch tested?

manual tests

@SparkQA
Copy link

SparkQA commented Sep 20, 2018

Test build #96351 has finished for PR 22491 at commit 5bb5f08.

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

# Conflicts:
#	sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/UnsafeArrayDataBenchmark.scala
@SparkQA
Copy link

SparkQA commented Sep 22, 2018

Test build #96463 has finished for PR 22491 at commit 1f492ee.

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

@wangyum
Copy link
Member Author

wangyum commented Sep 22, 2018

retest this please

@SparkQA
Copy link

SparkQA commented Sep 22, 2018

Test build #96467 has finished for PR 22491 at commit 1f492ee.

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

@wangyum
Copy link
Member Author

wangyum commented Sep 22, 2018

retest this please

@SparkQA
Copy link

SparkQA commented Sep 22, 2018

Test build #96468 has finished for PR 22491 at commit 1f492ee.

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

@wangyum
Copy link
Member Author

wangyum commented Sep 24, 2018

@dongjoon-hyun This refactor is ready to go. Thanks.

@dongjoon-hyun
Copy link
Member

Thank you for pinging me, @wangyum .

@dongjoon-hyun
Copy link
Member

Retest this please.

@SparkQA
Copy link

SparkQA commented Oct 2, 2018

Test build #96862 has finished for PR 22491 at commit 1f492ee.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyun
Copy link
Member

Could you review and merge wangyum#14 ?

@SparkQA
Copy link

SparkQA commented Oct 3, 2018

Test build #96878 has finished for PR 22491 at commit 5c819fb.

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

@SparkQA
Copy link

SparkQA commented Oct 3, 2018

Test build #96883 has finished for PR 22491 at commit 4b69423.

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

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM.

@dongjoon-hyun
Copy link
Member

Merged to master.

@asfgit asfgit closed this in 56741c3 Oct 3, 2018
@dongjoon-hyun
Copy link
Member

Thank you, @wangyum !

jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
## What changes were proposed in this pull request?

Refactor `UnsafeArrayDataBenchmark` to use main method.
Generate benchmark result:
```sh
SPARK_GENERATE_BENCHMARK_FILES=1 build/sbt "sql/test:runMain org.apache.spark.sql.execution.benchmark.UnsafeArrayDataBenchmark"
```

## How was this patch tested?

manual tests

Closes apache#22491 from wangyum/SPARK-25483.

Lead-authored-by: Yuming Wang <yumwang@ebay.com>
Co-authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
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