leventov commented on issue #7571: Optimize coordinator API to retrieve segments with overshadowed status URL: https://github.com/apache/incubator-druid/issues/7571#issuecomment-488332781 Probably all fields of `DataSegment` (except `overshadowed`) should remain final to avoid problematic concurrency updates when actually discovering an instance of #6358 problem (which should be very rare), but the general idea is the same: we should "attach less" to `DataSegment`'s immutability, therefore, I don't see that having a single mutable field in it would be a problem.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
