rzo1 opened a new pull request, #8986:
URL: https://github.com/apache/storm/pull/8986

   `nimbus.scheduler.strategy.class.whitelist` had no entry in 
`conf/defaults.yaml`, and `ReflectionUtils.newSchedulerStrategyInstance` 
treated an unset list as "allow any class", so Nimbus instantiated whatever 
class a topology named in `topology.scheduler.strategy`.
   
   Unset now means the six strategies that ship with Storm. An explicitly 
configured list, including an empty one, is honoured unchanged.
   
   Behaviour change: a cluster running an out-of-tree strategy with no 
allowlist configured will start failing scheduling for those topologies. The 
exception previously carried no message at all and now names the refused class 
and the config key to add. Adds `ReflectionUtilsTest`.


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

Reply via email to