gianm commented on issue #7233: Set "is_published" to false for overshadowed 
segments in sys.segments table
URL: 
https://github.com/apache/incubator-druid/issues/7233#issuecomment-476920684
 
 
   > is_overshadowed is defined as "overshadowed by other published segments 
that are loaded in the cluster"
   
   Any particular reason you’re thinking it makes sense to define 
`is_overshadowed` as “overshadowed by other published segments that are 
loaded”? (I assume this would mean: segments with used = true and that are 
available on some server)
   
   I’m thinking “overshadowed by some published segments” would be more useful 
in a debugging scenario. It allows you to write `is_published && 
!is_overshadowed` to mean “should be available”. In turn, comparing that to 
`is_available` helps narrow down where the current cluster state (what’s 
available) is different from the desired cluster state (what should be 
available).

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to