-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Jetty 12.1.x Infinispan flaky tests #12338
base: jetty-12.1.x
Are you sure you want to change the base?
Conversation
olamy
commented
Oct 1, 2024
- use Infinispan image 15.0.9, use UUID for cache name
- checkstyle
- use a single instance via singleton holder
- reduce memory usage of Infinispan
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of questions, then a couple of things to just tidy up.
} | ||
|
||
} | ||
|
||
public RemoteInfinispanTestSupport(String cacheName) | ||
{ | ||
Objects.requireNonNull(cacheName, "cacheName cannot be null"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the cacheName
now no longer used?
...n-common/src/main/java/org/eclipse/jetty/session/test/tools/RemoteInfinispanTestSupport.java
Outdated
Show resolved
Hide resolved
...n-common/src/main/java/org/eclipse/jetty/session/test/tools/RemoteInfinispanTestSupport.java
Outdated
Show resolved
Hide resolved
private static final GenericContainer<?> infinispan = new GenericContainer<>(IMAGE_NAME); | ||
|
||
static | ||
private static final class Holder |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the benefit of doing it this way vs having static
s?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
easier to read :)
@janbartel still a draft and working on it |
Signed-off-by: Olivier Lamy <[email protected]>
Signed-off-by: Olivier Lamy <[email protected]>
Signed-off-by: Olivier Lamy <[email protected]>
Signed-off-by: Olivier Lamy <[email protected]>
Signed-off-by: Olivier Lamy <[email protected]>
Signed-off-by: Olivier Lamy <[email protected]>
Signed-off-by: Olivier Lamy <[email protected]>
Signed-off-by: Olivier Lamy <[email protected]>
85ba257
to
489dca0
Compare