[GitHub] [druid] suneet-s commented on pull request #10900: Add query granularity to compaction task

[GitBox <gi...@apache.org>]

suneet-s commented on pull request #10900:
URL: https://github.com/apache/druid/pull/10900#issuecomment-789040463


   Assuming the doc changes are coming in a follow up change - we should make it clear what the tradeoffs are for rolling up data. 
   
   If someone accidentally rolls up data to a coarser query granularity (MONTH -> YEAR) - do they have any way to get the segments with the finer queryGranularity back (MONTH)? 
   
   > Note that the Compaction task ultimately still converts CompactionGranularitySpec to a UniformGranularitySpec when creating the index ingestion spec
   
   I don't understand the granularity specs enough. What's the impact of using UniformGranularitySpec instead of ArbitraryGranularitySpec? Is this something a user should be aware of?


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org