Skip to content

[SPARK-57682][BUILD] Add javax.servlet-api to LICENSE-binary#56763

Closed
dongjoon-hyun wants to merge 1 commit into
apache:masterfrom
dongjoon-hyun:SPARK-57682
Closed

[SPARK-57682][BUILD] Add javax.servlet-api to LICENSE-binary#56763
dongjoon-hyun wants to merge 1 commit into
apache:masterfrom
dongjoon-hyun:SPARK-57682

Conversation

@dongjoon-hyun

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This PR aims to add javax.servlet-api to LICENSE-binary.

Why are the changes needed?

Apache Spark 4.0.0+ distributions bundles javax.servlet-api-*.jar which is missed in LICENSE-binary.

Does this PR introduce any user-facing change?

No behavior change.

How was this patch tested?

Manual review.

Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Claude Opus 4.8)

@dongjoon-hyun

dongjoon-hyun commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

cc @huaxingao as the release manager of Apache Spark 4.2.0.

@huaxingao huaxingao left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@dongjoon-hyun

Copy link
Copy Markdown
Member Author

Thank you, @huaxingao !

dongjoon-hyun added a commit that referenced this pull request Jun 25, 2026
### What changes were proposed in this pull request?

This PR aims to add `javax.servlet-api` to `LICENSE-binary`.

### Why are the changes needed?

Apache Spark 4.0.0+ distributions bundles `javax.servlet-api-*.jar` which is missed in `LICENSE-binary`.

- #45154

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

No behavior change.

### How was this patch tested?

Manual review.

### Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Claude Opus 4.8)

Closes #56763 from dongjoon-hyun/SPARK-57682.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
(cherry picked from commit 994f049)
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
dongjoon-hyun added a commit that referenced this pull request Jun 25, 2026
### What changes were proposed in this pull request?

This PR aims to add `javax.servlet-api` to `LICENSE-binary`.

### Why are the changes needed?

Apache Spark 4.0.0+ distributions bundles `javax.servlet-api-*.jar` which is missed in `LICENSE-binary`.

- #45154

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

No behavior change.

### How was this patch tested?

Manual review.

### Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Claude Opus 4.8)

Closes #56763 from dongjoon-hyun/SPARK-57682.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
(cherry picked from commit 994f049)
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
dongjoon-hyun added a commit that referenced this pull request Jun 25, 2026
### What changes were proposed in this pull request?

This PR aims to add `javax.servlet-api` to `LICENSE-binary`.

### Why are the changes needed?

Apache Spark 4.0.0+ distributions bundles `javax.servlet-api-*.jar` which is missed in `LICENSE-binary`.

- #45154

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

No behavior change.

### How was this patch tested?

Manual review.

### Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Claude Opus 4.8)

Closes #56763 from dongjoon-hyun/SPARK-57682.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
(cherry picked from commit 994f049)
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
dongjoon-hyun added a commit that referenced this pull request Jun 25, 2026
### What changes were proposed in this pull request?

This PR aims to add `javax.servlet-api` to `LICENSE-binary`.

### Why are the changes needed?

Apache Spark 4.0.0+ distributions bundles `javax.servlet-api-*.jar` which is missed in `LICENSE-binary`.

- #45154

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

No behavior change.

### How was this patch tested?

Manual review.

### Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Claude Opus 4.8)

Closes #56763 from dongjoon-hyun/SPARK-57682.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
(cherry picked from commit 994f049)
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
@dongjoon-hyun

Copy link
Copy Markdown
Member Author

Merged to master/4.x/4.2/4.1/4.0.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-57682 branch June 25, 2026 06:49
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.

2 participants