Skip to content

[SPARK-32150][BUILD] Upgrade to ZStd 1.4.5-4#28969

Closed
williamhyun wants to merge 1 commit into
apache:masterfrom
williamhyun:zstd2
Closed

[SPARK-32150][BUILD] Upgrade to ZStd 1.4.5-4#28969
williamhyun wants to merge 1 commit into
apache:masterfrom
williamhyun:zstd2

Conversation

@williamhyun

@williamhyun williamhyun commented Jul 1, 2020

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This PR aims to upgrade to ZStd 1.4.5-4.

Why are the changes needed?

ZStd 1.4.5-4 fixes the following.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass the Jenkins.

@williamhyun williamhyun changed the title [SPARK-XXX][BUILD] Upgrade to ZStd 1.4.5-4 [SPARK-32150][BUILD] Upgrade to ZStd 1.4.5-4 Jul 1, 2020
@SparkQA

SparkQA commented Jul 1, 2020

Copy link
Copy Markdown

Test build #124805 has finished for PR 28969 at commit 7abf2b5.

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

@maropu

maropu commented Jul 2, 2020

Copy link
Copy Markdown
Member

Looks okay to me. cc: @dongjoon-hyun @srowen

@srowen

srowen commented Jul 2, 2020

Copy link
Copy Markdown
Member

Looks OK to backport to 3.0.x right?

@maropu

maropu commented Jul 2, 2020

Copy link
Copy Markdown
Member

branch-3.0 still uses zstd v1.4.4 because we didn't merge the last zstd upgrade commit (#28682) into branch-3.0.

// zstd version in branch-3.0

spark/pom.xml

Line 668 in bdbfe6e

<version>1.4.4-3</version>

If there is no compatiblity issue and the fixes are somewhat critical, I think merging this into branch-3.0 looks fine.

@SparkQA

SparkQA commented Jul 2, 2020

Copy link
Copy Markdown

Test build #124821 has finished for PR 28969 at commit 7abf2b5.

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

@maropu

maropu commented Jul 2, 2020

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 2, 2020

Copy link
Copy Markdown

Test build #124843 has finished for PR 28969 at commit 7abf2b5.

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

@maropu

maropu commented Jul 2, 2020

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 2, 2020

Copy link
Copy Markdown

Test build #124844 has finished for PR 28969 at commit 7abf2b5.

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

@SparkQA

SparkQA commented Jul 2, 2020

Copy link
Copy Markdown

Test build #124845 has finished for PR 28969 at commit 7abf2b5.

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

@dongjoon-hyun dongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1, LGTM.

@SparkQA

SparkQA commented Jul 2, 2020

Copy link
Copy Markdown

Test build #124914 has finished for PR 28969 at commit 7abf2b5.

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

@SparkQA

SparkQA commented Jul 2, 2020

Copy link
Copy Markdown

Test build #124918 has finished for PR 28969 at commit 7abf2b5.

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

@maropu

maropu commented Jul 2, 2020

Copy link
Copy Markdown
Member

retest this please

1 similar comment
@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 5, 2020

Copy link
Copy Markdown

Test build #124934 has started for PR 28969 at commit 7abf2b5.

@maropu

maropu commented Jul 6, 2020

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 6, 2020

Copy link
Copy Markdown

Test build #125041 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 6, 2020

Copy link
Copy Markdown

Test build #125062 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 6, 2020

Copy link
Copy Markdown

Test build #125069 has started for PR 28969 at commit 7abf2b5.

@SparkQA

SparkQA commented Jul 6, 2020

Copy link
Copy Markdown

Test build #125133 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 7, 2020

Copy link
Copy Markdown

Test build #125154 has finished for PR 28969 at commit 7abf2b5.

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

@SparkQA

SparkQA commented Jul 8, 2020

Copy link
Copy Markdown

Test build #125396 has finished for PR 28969 at commit 7abf2b5.

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

@maropu

maropu commented Jul 8, 2020

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 9, 2020

Copy link
Copy Markdown

Test build #125418 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 9, 2020

Copy link
Copy Markdown

Test build #125430 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 9, 2020

Copy link
Copy Markdown

Test build #125438 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 9, 2020

Copy link
Copy Markdown

Test build #125444 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 9, 2020

Copy link
Copy Markdown

Test build #125467 has started for PR 28969 at commit 7abf2b5.

@SparkQA

SparkQA commented Jul 9, 2020

Copy link
Copy Markdown

Test build #125504 has finished for PR 28969 at commit 7abf2b5.

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

@SparkQA

SparkQA commented Jul 10, 2020

Copy link
Copy Markdown

Test build #125519 has finished for PR 28969 at commit 7abf2b5.

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

@maropu

maropu commented Jul 10, 2020

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 10, 2020

Copy link
Copy Markdown

Test build #125541 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 10, 2020

Copy link
Copy Markdown

Test build #125560 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 10, 2020

Copy link
Copy Markdown

Test build #125575 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 10, 2020

Copy link
Copy Markdown

Test build #125607 has finished for PR 28969 at commit 7abf2b5.

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

@SparkQA

SparkQA commented Jul 10, 2020

Copy link
Copy Markdown

Test build #125629 has finished for PR 28969 at commit 7abf2b5.

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

@SparkQA

SparkQA commented Jul 11, 2020

Copy link
Copy Markdown

Test build #125652 has finished for PR 28969 at commit 7abf2b5.

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

@maropu

maropu commented Jul 11, 2020

Copy link
Copy Markdown
Member

retest this please

@SparkQA

SparkQA commented Jul 11, 2020

Copy link
Copy Markdown

Test build #125673 has finished for PR 28969 at commit 7abf2b5.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

Merged to master.

holdenk pushed a commit to holdenk/spark that referenced this pull request Oct 27, 2020
### What changes were proposed in this pull request?

This PR aims to upgrade to ZStd 1.4.5-4.

### Why are the changes needed?

ZStd 1.4.5-4 fixes the following.

- luben/zstd-jni@3d16e51
- luben/zstd-jni@3d51bdc
### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Pass the Jenkins.

Closes apache#28969 from williamhyun/zstd2.

Authored-by: William Hyun <williamhyun3@gmail.com>
Signed-off-by: HyukjinKwon <gurwls223@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants