[GitHub] [druid] a2l007 commented on pull request #10070: Fix balancer strategy

2020-06-30 Thread GitBox
a2l007 commented on pull request #10070: URL: https://github.com/apache/druid/pull/10070#issuecomment-652068807 I agree. However, the docs give an impression that it maybe okay to set this to the default value, although it isn't. Have tried to fix this in the linked PR. -

[GitHub] [druid] a2l007 commented on pull request #10070: Fix balancer strategy

2020-06-30 Thread GitBox
a2l007 commented on pull request #10070: URL: https://github.com/apache/druid/pull/10070#issuecomment-652063436 I feel we should mention this in the release notes since it modifies the default behavior for `druid.server.maxSize`. For clusters having historicals with this property set to ze