#7274 Reduce mem consumption from reference queue

[Iapetus999]

#7274

An alternative would be to create a env flag for this.

[linux-foundation-easycla]

• ❌ - login: @Iapetus999 / name: Andrew Rosenberg . The commit (1ca9999) is not authorized under a signed CLA. Please click here to be authorized. For further assistance with EasyCLA, please submit a support request ticket.

[trask]

hi @lapetus999! check out #6240, I believe it's addressing the same issue, and probably we should get that merged?

[Iapetus999]

Yeah we've been having a lot of problems with this. I'm okay with closing this as a dupe of the other PR. What needs to be done to get the other PR merged?

[trask]

Yeah we've been having a lot of problems with this. I'm okay with closing this as a dupe of the other PR. What needs to be done to get the other PR merged?

great, since you have an active repro, can you build and test that PR to confirm things look good for you?

[Iapetus999]

Since the issue only repros in Production, I won't be able to test it until next week (holiday code freeze). I am running this PR and it's looking good.

[trask]

Since the issue only repros in Production, I won't be able to test it until next week (holiday code freeze)

that would be great, the next release (1.21.0) is targeted for December 14

[trask]

I think we can close this, now that #6240 has been merged, thx for raising this!

It would still be really helpful if you could report back your findings after you have a chance to test the latest snapshot build: https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/CONTRIBUTING.md#snapshot-builds

[Iapetus999]

Hi @trask
We've had this running in Production for a few days now and it's looking good.
I grabbed opentelemetry-javaagent-1.21.0-20221206.081659-86.jar

[trask]

@Iapetus999 thx! 1.21.0 release should be out in a couple of days