jihoonson commented on pull request #9905:
URL: https://github.com/apache/druid/pull/9905#issuecomment-631854837


   If I'm reading this change correctly, the compaction task will read both old 
and new segments for those overlapping intervals. This doesn't seem right to me 
since the new segment should overshadow the old segment, which means, the 
compaction task should read only the new segment for the overlapping intervals.
   
   Can you share some more details of your use case? Why do you want to compact 
segments of different versions together?


----------------------------------------------------------------
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]



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

Reply via email to