Skip to content

Conversation

@wangyum
Copy link
Member

@wangyum wangyum commented Dec 13, 2019

What changes were proposed in this pull request?

This PR add support do not add commits to master branch when releasing preview version.

Why are the changes needed?

We need manual revert this change, example:
image

Does this PR introduce any user-facing change?

No.

How was this patch tested?

manual test

@SparkQA
Copy link

SparkQA commented Dec 13, 2019

Test build #115294 has finished for PR 26879 at commit 0b75e4d.

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

Copy link
Member

@srowen srowen left a comment

Choose a reason for hiding this comment

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

Seems fine if it still works.

@dongjoon-hyun
Copy link
Member

dongjoon-hyun commented Dec 13, 2019

Sorry guys. I'm -1 for this.

@dongjoon-hyun
Copy link
Member

dongjoon-hyun commented Dec 13, 2019

IMO, it's just two commits which are virtually harmless.


if ! is_dry_run; then
# Push changes
git push origin $RELEASE_TAG
Copy link
Member

@dongjoon-hyun dongjoon-hyun Dec 13, 2019

Choose a reason for hiding this comment

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

Where is this tag pointing at, @wangyum ?

Copy link
Member

Choose a reason for hiding this comment

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

I'm wondering if we can have a pointer during vote like the following.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes. We still have a similar pointer. But we do not add any commits.

Copy link
Member

Choose a reason for hiding this comment

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

😮 It's interesting. Does the pointer look like the following really?

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

v3.0.0-preview2-rc1: 1b41cb5

@dongjoon-hyun
Copy link
Member

cc @gatorsmile

@SparkQA
Copy link

SparkQA commented Dec 15, 2019

Test build #115348 has finished for PR 26879 at commit c7cd6f9.

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

@wangyum
Copy link
Member Author

wangyum commented Dec 16, 2019

I'm merging it and will be releasing preview2 soon.

@wangyum wangyum closed this in 26b658f Dec 16, 2019
@wangyum
Copy link
Member Author

wangyum commented Dec 16, 2019

Merged to master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants