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

[UNDERTOW-2385] Modify ThreadLocalCache using WeakHashMap #1589

Closed
wants to merge 2 commits into from

Conversation

ddaaac
Copy link

@ddaaac ddaaac commented May 3, 2024

@baranowb baranowb added the waiting CI check Ready to be merged but waiting for CI check label May 6, 2024
@baranowb
Copy link
Contributor

baranowb commented May 6, 2024

LGTM

@baranowb baranowb self-requested a review May 6, 2024 06:01
@@ -24,11 +24,7 @@

import java.lang.ref.WeakReference;
import java.nio.ByteBuffer;
import java.util.ArrayDeque;
Copy link
Contributor

Choose a reason for hiding this comment

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

Error: /home/runner/work/undertow/undertow/core/src/main/java/io/undertow/server/DefaultByteBufferPool.java:27: Using the '.' form of import should be avoided - java.util.. [AvoidStarImport]

@baranowb baranowb added waiting PR update Awaiting PR update(s) from contributor before merging failed CI Introduced new regession(s) during CI check labels May 6, 2024
@ddaaac
Copy link
Author

ddaaac commented May 7, 2024

What is the next release date? If the date is far away, we will try to downgrade.

@fl4via
Copy link
Member

fl4via commented Jun 10, 2024

we are releasing this week @ddaaac

@fl4via
Copy link
Member

fl4via commented Jun 10, 2024

@ddaaac I am merging this one with #1591 so we can have the weak hash map synchronized. Please see #1591. I am closing this one and marking it as incorporated

@fl4via fl4via added incorporated Incorporated in another PR and removed waiting PR update Awaiting PR update(s) from contributor before merging waiting CI check Ready to be merged but waiting for CI check failed CI Introduced new regession(s) during CI check labels Jun 10, 2024
@fl4via fl4via closed this Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
incorporated Incorporated in another PR
Projects
None yet
3 participants