[GitHub] gianm opened a new pull request #6652: Fix missing default config in some calls to coordinator dynamic configs.

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

gianm opened a new pull request #6652: Fix missing default config in some calls to coordinator dynamic configs.
URL: https://github.com/apache/incubator-druid/pull/6652
 
 
   The lack of a default config meant that if someone called an API
   _without_ a default config before one _with_ a default config, then
   the default value would get stuck at null instead of the intended
   default value. I noticed this in a cluster where calling /druid/coordinator/v1/config
   before a coordinator had fully started up would lead to NPEs during
   DruidCoordinatorRuleRunner.
   
   This patch makes the default configs consistent across all calls.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

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