1berry/openjdk
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

8354460: Streaming output for attach API should be turned on by default

Browse Source 
Reviewed-by: sspitsyn, kevinw
This commit is contained in:
Alex Menkov 2025-06-03 20:47:17 +00:00
parent 704b5990a7
commit da49fa5e15
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/hotspot/share/services/attachListener.cpp
View File

@ -172,8 +172,8 @@ volatile AttachListenerState AttachListener::_state = AL_NOT_INITIALIZED;
AttachAPIVersion AttachListener::_supported_version = ATTACH_API_V1; AttachAPIVersion AttachListener::_supported_version = ATTACH_API_V1;
// Default is false (if jdk.attach.vm.streaming property is not set). // Default is true (if jdk.attach.vm.streaming property is not set).
bool AttachListener::_default_streaming_output = false; bool AttachListener::_default_streaming_output = true;
static bool get_bool_sys_prop(const char* name, bool default_value, TRAPS) { static bool get_bool_sys_prop(const char* name, bool default_value, TRAPS) {
ResourceMark rm(THREAD); ResourceMark rm(THREAD);

3
src/jdk.attach/share/classes/sun/tools/attach/HotSpotVirtualMachine.java
View File

@ -63,9 +63,8 @@ public abstract class HotSpotVirtualMachine extends VirtualMachine {
static { static {
String s = VM.getSavedProperty("jdk.attach.allowAttachSelf"); String s = VM.getSavedProperty("jdk.attach.allowAttachSelf");
ALLOW_ATTACH_SELF = "".equals(s) || Boolean.parseBoolean(s); ALLOW_ATTACH_SELF = "".equals(s) || Boolean.parseBoolean(s);
// For now the default is false.
String s2 = VM.getSavedProperty("jdk.attach.allowStreamingOutput"); String s2 = VM.getSavedProperty("jdk.attach.allowStreamingOutput");
ALLOW_STREAMING_OUTPUT = "".equals(s2) || Boolean.parseBoolean(s2); ALLOW_STREAMING_OUTPUT = !("false".equals(s2));
} }
private final boolean selfAttach; private final boolean selfAttach;