Skip to content

Conversation

@beliefer
Copy link
Contributor

@beliefer beliefer commented Oct 8, 2019

What changes were proposed in this pull request?

The LICENSE file exists a minor issue has an incorrect path.
This PR will fix it.

Why are the changes needed?

This is a minor bug.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Exists UT.

@HyukjinKwon
Copy link
Member

Merged to master and branch-2.4.

HyukjinKwon pushed a commit that referenced this pull request Oct 8, 2019
### What changes were proposed in this pull request?
The `LICENSE` file exists a minor issue has an incorrect path.
This PR will fix it.

### Why are the changes needed?
This is a minor bug.

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

### How was this patch tested?
Exists UT.

Closes #26050 from beliefer/resolve-minor-license-issue.

Authored-by: gengjiaan <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
@beliefer
Copy link
Contributor Author

beliefer commented Oct 8, 2019

@HyukjinKwon Thanks for your help.

@SparkQA
Copy link

SparkQA commented Oct 8, 2019

Test build #111873 has finished for PR 26050 at commit dc45811.

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

atronchi pushed a commit to atronchi/spark that referenced this pull request Oct 23, 2019
### What changes were proposed in this pull request?
The `LICENSE` file exists a minor issue has an incorrect path.
This PR will fix it.

### Why are the changes needed?
This is a minor bug.

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

### How was this patch tested?
Exists UT.

Closes apache#26050 from beliefer/resolve-minor-license-issue.

Authored-by: gengjiaan <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
srowen pushed a commit that referenced this pull request Nov 13, 2019
### What changes were proposed in this pull request?
I want to say sorry! this PR follows the previous #26050.
I didn't find them at the same time.
The `LICENSE-binary` file exists a minor issue has an incorrect path.
This PR will fix it.

### Why are the changes needed?
This is a minor bug.

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

### How was this patch tested?
Exists UT.

Closes #26490 from beliefer/fix-minor-license-issue.

Authored-by: gengjiaan <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
@beliefer beliefer deleted the resolve-minor-license-issue branch December 6, 2019 10:09
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