# HG changeset patch # User egahlin # Date 1567042069 -7200 # Node ID 08327ab323c54cba6e9c08ef0a73b0c20caa35fa # Parent de358b994ad92933945bf82ed296fee8352f51ea Make stream thread non-daemon and remove unnecessary copying of filter diff -r de358b994ad9 -r 08327ab323c5 src/jdk.jfr/share/classes/jdk/jfr/consumer/AbstractEventStream.java --- a/src/jdk.jfr/share/classes/jdk/jfr/consumer/AbstractEventStream.java Thu Aug 29 03:21:55 2019 +0200 +++ b/src/jdk.jfr/share/classes/jdk/jfr/consumer/AbstractEventStream.java Thu Aug 29 03:27:49 2019 +0200 @@ -187,7 +187,7 @@ } ef.setThreshold(name, 0); } - return ef.threadSafe(); + return ef; } final public StreamConfiguration setReuse(boolean reuse) { @@ -342,7 +342,6 @@ // Create thread object in constructor to ensure caller has // permission before constructing object thread = new Thread(this); - thread.setDaemon(true); } public final void run() { diff -r de358b994ad9 -r 08327ab323c5 src/jdk.jfr/share/classes/jdk/jfr/internal/consumer/InternalEventFilter.java --- a/src/jdk.jfr/share/classes/jdk/jfr/internal/consumer/InternalEventFilter.java Thu Aug 29 03:21:55 2019 +0200 +++ b/src/jdk.jfr/share/classes/jdk/jfr/internal/consumer/InternalEventFilter.java Thu Aug 29 03:27:49 2019 +0200 @@ -39,12 +39,6 @@ this(false, new HashMap<>()); } - // returns an instance that can be passed to - // another thread safely - public InternalEventFilter threadSafe() { - return new InternalEventFilter(acceptAll, thresholds); - } - private InternalEventFilter(boolean acceptAll, Map thresholds) { this.acceptAll = acceptAll; this.thresholds = thresholds;