Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the shading issue for javax security #18455

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

beinan
Copy link
Contributor

@beinan beinan commented Dec 1, 2023

What changes are proposed in this pull request?

Fix the shading issue for javax security

Why are the changes needed?

you might see class not found in javax's security module

Does this PR introduce any user facing changes?

no

@JiamingMai JiamingMai self-requested a review December 4, 2023 08:01
@JiamingMai JiamingMai added the type-bug This issue is about a bug label Dec 4, 2023
Copy link
Contributor

@JiamingMai JiamingMai left a comment

Choose a reason for hiding this comment

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

LGTM

@beinan
Copy link
Contributor Author

beinan commented Dec 4, 2023

alluxio-bot, merge this please.

@alluxio-bot alluxio-bot merged commit 0db1ff7 into main Dec 4, 2023
12 checks passed
ssz1997 pushed a commit to ssz1997/alluxio that referenced this pull request Dec 15, 2023
### What changes are proposed in this pull request?

Fix the shading issue for javax security

### Why are the changes needed?
you might see class not found in javax's security module

### Does this PR introduce any user facing changes?
no

			pr-link: Alluxio#18455
			change-id: cid-d04ba461930b7abba2613810e8d7d2ae77d7e3be
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug This issue is about a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants