Skip to content

Commit cf35e35

Browse files
anuraagalaurit
andauthored
Use type expressions from Java 17 (#207)
* Use type expressions from Java 17 * Update jfr-streaming/src/main/java/io/opentelemetry/contrib/jfr/metrics/internal/memory/GCHeapSummaryHandler.java Co-authored-by: Lauri Tulmin <[email protected]> Co-authored-by: Lauri Tulmin <[email protected]>
1 parent 89aa698 commit cf35e35

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

jfr-streaming/src/main/java/io/opentelemetry/contrib/jfr/metrics/internal/ThreadGrouper.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ public final class ThreadGrouper {
1313
// FIXME doesn't actually do any grouping, but should be safe for now
1414
public Optional<String> groupedName(RecordedEvent ev) {
1515
Object thisField = ev.getValue("eventThread");
16-
if (thisField instanceof RecordedThread) {
17-
return Optional.of(((RecordedThread) thisField).getJavaName());
16+
if (thisField instanceof RecordedThread thread) {
17+
return Optional.of(thread.getJavaName());
1818
}
1919
return Optional.empty();
2020
}

jfr-streaming/src/main/java/io/opentelemetry/contrib/jfr/metrics/internal/memory/GCHeapSummaryHandler.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,9 +105,7 @@ private void recordValues(RecordedEvent before, RecordedEvent after) {
105105
memoryHistogram.record(after.getLong(HEAP_USED), ATTR_MEMORY_USED);
106106
}
107107
if (after.hasField(HEAP_SPACE)) {
108-
after.getValue(HEAP_SPACE);
109-
if (after.getValue(HEAP_SPACE) instanceof RecordedObject) {
110-
RecordedObject ro = after.getValue(HEAP_SPACE);
108+
if (after.getValue(HEAP_SPACE) instanceof RecordedObject ro) {
111109
memoryHistogram.record(ro.getLong(COMMITTED_SIZE), ATTR_MEMORY_COMMITTED);
112110
}
113111
}

0 commit comments

Comments
 (0)