Made `SegmentMetadataHolder` immutable, and it seems `segmentMetadataInfo` can be a HashMap now protected by `lock`. Yes `getSegmentMetadata()` is also returning `Map<DataSegment, SegmentMetadataHolder>` created under the `lock`
[ Full content available at: https://github.com/apache/incubator-druid/pull/6094 ] This message was relayed via gitbox.apache.org for devnull@infra.apache.org