[GitHub] [druid] capistrant commented on issue #11161: Need for a more elegant upgrade path for new Dynamic Configs

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

capistrant commented on issue #11161:
URL: https://github.com/apache/druid/issues/11161#issuecomment-827124224


   The gotcha seems to be that the Builder is not used with defaults when we are serializing from what is in metastore. Someone who is starting with a fresh deployment would get the new default for the added config. But someone just doing an upgrade doesn't benefit from the defaults in the Builder class


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