Github user neykov commented on the issue:

    https://github.com/apache/brooklyn-server/pull/573
  
    The changes include unrelated change in the class loading fallbacks - 
increasing the priority of the bundle prefix in relation to the other 
fallbacks. Contrary to the comment in 
https://github.com/apache/brooklyn-server/pull/573/commits/40330602af958d1e3486feea81933168d44e936d,
 the changes include behaviour changes, not only logging improvements. Would've 
been useful to have it in a separate commit + PR for better visibility if we 
ever need to come back to the changes.
    The `RedisClusterIntegrationTest` would've been better added to the 
`brooklyn-camp` project which is the place to add CAMP tests which don't fit 
elsewhere - we really need to split them in an camp-integration-test project.
    Comments above are not worth blocking the merge. Merging now.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to