-
Notifications
You must be signed in to change notification settings - Fork 580
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
CRaC support #9643
base: main
Are you sure you want to change the base?
CRaC support #9643
Conversation
faf73cb
to
58ea364
Compare
1492a5d
to
5077028
Compare
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.
Virtual threads metrics changes LGTM.
This comment was marked as resolved.
This comment was marked as resolved.
a0c9219
to
a64e04a
Compare
cf33d05
to
e1f6b3b
Compare
webserver/webserver/src/main/java/io/helidon/webserver/LoomServer.java
Outdated
Show resolved
Hide resolved
webserver/webserver/src/main/java/io/helidon/webserver/ServerListener.java
Show resolved
Hide resolved
...grations/cdi/jta-cdi/src/main/java/io/helidon/integrations/jta/cdi/NarayanaCracResource.java
Outdated
Show resolved
Hide resolved
integrations/cdi/jta-cdi/src/main/java/io/helidon/integrations/jta/narayana/ProcessId.java
Outdated
Show resolved
Hide resolved
5c2592b
to
421cf68
Compare
Signed-off-by: Radim Vansa <[email protected]>
421cf68
to
7517efd
Compare
Signed-off-by: Daniel Kec <[email protected]>
7517efd
to
d27ad4a
Compare
* @return uptime of the Java virtual machine in milliseconds. | ||
* @see java.lang.management.RuntimeMXBean#getStartTime() | ||
*/ | ||
long getUptimeSinceResume(); |
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.
Use Helidon style ? I.e. uptimeSinceResume()
* @return start time of the Java virtual machine in milliseconds. | ||
* @see java.lang.management.RuntimeMXBean#getUptime() | ||
*/ | ||
long getResumeTime(); |
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.
Use Helidon style ? I.e. resumeTime()
* Returns either uptime since resume, or since JVM start if not resumed. | ||
* @return actual uptime in milliseconds | ||
*/ | ||
default long getUptime() { |
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.
Use Helidon style ? I.e. uptime()
Fixes #4843 continuation of #8743
Integration works as a feature, when dependency is on the classpath:
Helidon modules are CRaC aware and able to suspend features blocking successful snapshot creation.