in AbstractApplication, don't infer locations to pass to start(...)

with this, locations defined on an app are no longer inserted automatically by BasicApplication.start().  this is because locations are now normally defined on yaml.  entities typically prefer a location if supplied to start(...), but if none are supplied there they look in local configuration and then ancestor configuration.  if BasicApplication passes their configured location to the start method, that is trumping any location defined on they descendant entity, which is probably not desired.  if an entity does not define a location, there is no change to behaviour as it looks up ancestor location regardless.  but this fixes a bug where if a location A is defined on an application, and another location B defined on a particular entity in that application, the entity now uses B whereas previously it was using A.  this makes explicit entity startup behaviour consistent with how an entity spec used in a cluster works (which has and still prefers the location defined in the entity spec).
1 file changed
tree: 4c731759564a1df499ce5c89d9953fce2008272d
  1. brooklyn/
  2. brooklyn-dist/
  3. brooklyn-docs/
  4. brooklyn-library/
  5. brooklyn-server/
  6. brooklyn-ui/
  7. docs/
  8. usage/
  9. .gitattributes
  10. .gitignore
  11. LICENSE
  12. NOTICE
  13. pom.xml
  14. README.md
README.md

Brooklyn

Apache Brooklyn Incubator Historic Repo

This is the historical incubator repo for Apache Brooklyn.

Brooklyn has graduated. Visit us here.

The sub-directories in this project correspond to multiple separate repositories in apache. (The link above to brooklyn/ started life exactly as a copy of brooklyn/ in this folder, as an uber-project for the others, including the server and the ui.)

To Build

The code can be built with a:

mvn clean install

This creates a build in usage/dist/target/brooklyn-dist. Run with bin/brooklyn launch.