Skip to content
This repository was archived by the owner on Sep 2, 2022. It is now read-only.
/ jdk16 Public archive

8257621: JFR StringPool misses cached items across consecutive recordings #18

Closed
wants to merge 8 commits into from
+106 −152
Closed

8257621: JFR StringPool misses cached items across consecutive recordings #18

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
c59c8f1
Reintroduce string pool generation
mgronlun Dec 13, 2020
d887127
javadoc update
mgronlun Dec 13, 2020
e903dc7
rephrasing
mgronlun Dec 13, 2020
b35983c
whitespace
mgronlun Dec 13, 2020
6cc4284
fallback on incorrect generation is the EventWriter notification
mgronlun Dec 15, 2020
6b7e5ab
remove generation concept
mgronlun Dec 16, 2020
ad69b84
whitespace
mgronlun Dec 16, 2020
4e1c32c
protect this.currentSizeUTF16 on reset
mgronlun Dec 16, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion src/jdk.jfr/share/classes/jdk/jfr/internal/StringPool.java
View file
Original file line number Diff line number Diff line change
@@ -65,7 +65,9 @@ private static class SimpleStringIdPool {

private void reset() {
this.cache.clear();
this.currentSizeUTF16 = 0;
synchronized(SimpleStringIdPool.class) {
this.currentSizeUTF16 = 0;
}
}

private long addString(String s) {