capistrant commented on PR #18939: URL: https://github.com/apache/druid/pull/18939#issuecomment-3924796895
thanks again for review round @clintropolis I think a lot makes sense and I've got most the changes locally. The segment granularity thing I'm at the point where it is so ripe for edge cases and errors where an operator may get themselves in trouble and regret it that I think the static set of standard granularities I called out in the comment is the way. my take is that it still gives people a lot more flexibility than operators have today, while trying to make our lives easier coding it, and prevent them (operators) from doing something crazy that they hate. what do you think? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
