This is essentially copying all entries of `DruidSchema.segmentMetadataInfo`. 
Then, why not just returning `ImmutableMap.copyOf(segmentMetadataInfo)` in 
`DruidSchema.getSegmentMetadataInfo()`? This would be better because the entire 
code for concurrent access to `segmentMetaInfo` can be handled in `DataSchema`.

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

Reply via email to