Skip to content

Conversation

@HeartSaVioR
Copy link
Contributor

What changes were proposed in this pull request?

This patch fixes the build failure in branch-3.0 due to cherry-picking SPARK-31045 to branch-3.0, as .version() is not available in branch-3.0 yet.

Why are the changes needed?

The build is failing in branch-3.0.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Jenkins build will verify.

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. Thank you, @HeartSaVioR .
Merged to branch-3.0. This is verified locally since this is urgent.

dongjoon-hyun pushed a commit that referenced this pull request Mar 6, 2020
… master and 3.0

### What changes were proposed in this pull request?

This patch fixes the build failure in `branch-3.0` due to cherry-picking SPARK-31045 to branch-3.0, as `.version()` is not available in `branch-3.0` yet.

### Why are the changes needed?

The build is failing in `branch-3.0`.

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

No.

### How was this patch tested?

Jenkins build will verify.

Closes #27826 from HeartSaVioR/SPARK-31045-branch-3.0-FOLLOWUP.

Authored-by: Jungtaek Lim (HeartSaVioR) <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
@HeartSaVioR
Copy link
Contributor Author

Thanks for the quick review and merge!

@HeartSaVioR HeartSaVioR deleted the SPARK-31045-branch-3.0-FOLLOWUP branch March 6, 2020 05:31
@cloud-fan
Copy link
Contributor

sorry to be late, thanks for fixing!

@SparkQA
Copy link

SparkQA commented Mar 6, 2020

Test build #119445 has finished for PR 27826 at commit 6c367ab.

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

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.

4 participants