The `brooklyn.entity.basic` package is getting very full. As we're adding more effector-related classes, do you think we should create a `brooklyn.entity.basic.effector` package?

If we did that, I'd like to see `EffectorTaskFactory` as a top-level interface (rather than nested).