Published segments cache in Brokers shouldn't use Interners

[leventov]

Regarding #6901 (proposal: #6834)

Published segments cache in Brokers shouldn't use Interners. Instead, MetadataSegmentView should probe into BrokerServerView's data and vice versa, similar to was is done in Coordinator's memory: #6357.

FYI @surekhasaharan

[gianm]

What's wrong with Interners? I don't see a discussion in #6357.

[leventov]

The explanation is in the PR description. You are getting rid of one giant map with weak references which hogs the memory. Weak refs are also bad for GC.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

This issue is no longer marked as stale.