search

grafana / jfr-parser

Java Flight Recorder parser library written in Go.
Apache License 2.0
43 stars 15 forks source link

Update IDMap to use uint64 #40

Closed korniltsev closed 2 weeks ago

korniltsev commented 2 weeks ago

https://github.com/grafana/jfr-parser/pull/39

sivachandran commented 2 weeks ago

As far as I searched, IDMap is only used in MethodList. MethodRef already uses uint64 which would be used as key for the IDMap. So I don't see a need to change anything further.

korniltsev commented 2 weeks ago

ok I'm wrong. Sorry for confusion.