The segments that are stored in 
`SQLMetadataSegmentManager.DATA_SEGMENT_INTERNER` are the same as in 
`SQLMetadataSegmentManager.dataSourcesRef`, so we could effectively use the 
latter for interning. It allows to get rid of one large Map of *weak 
references* which are probably bad for GC themselves.

Also interning doesn't make sense in `enableDatasource()`.

Originally interning was introduced in #3267.

[ Full content available at: 
https://github.com/apache/incubator-druid/pull/6357 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to