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

WFLY-19349 Add missing protostream marshaller (and Immutability) for SessionMap.Mutex #17896

Merged
merged 1 commit into from
Jun 13, 2024

Conversation

pferraro
Copy link
Contributor

@pferraro pferraro commented May 17, 2024

@pferraro
Copy link
Contributor Author

Closing for analysis...

@pferraro pferraro closed this May 17, 2024
@pferraro pferraro reopened this May 23, 2024
@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label May 24, 2024
@pferraro
Copy link
Contributor Author

pferraro commented May 24, 2024

Strange. org.glassfish:jakarta.faces:4.1.0 is missing from nexus (and maven central). Any ideas @jasondlee ?

UPDATE: This is because the release ballot has not yet complete. I'll add a hold label until then.
UPDATE: Face 4.1.0 was updated in preview via #17946

@pferraro pferraro added the hold PR should not be merged for some reason. label May 29, 2024
@pferraro pferraro removed the hold PR should not be merged for some reason. label Jun 13, 2024
import java.util.concurrent.atomic.AtomicReference;

import jakarta.servlet.ServletContext;
import jakarta.servlet.http.HttpSession;
Copy link
Contributor

Choose a reason for hiding this comment

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

Since I'm on an import nazi kick....

jakarta isn't special like java and javax

Copy link
Contributor

@jasondlee jasondlee left a comment

Choose a reason for hiding this comment

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

This looks fine to me.

@bstansberry bstansberry merged commit 3123530 into wildfly:main Jun 13, 2024
13 checks passed
@bstansberry
Copy link
Contributor

Thanks @pferraro and @jasondlee

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes
Projects
None yet
3 participants