GitHub user aledsage opened a pull request:

    https://github.com/apache/brooklyn-server/pull/864

    Separate dumper class

    Move all `Entities.dumpInfo` to `Dumper`, deprecating the methods in 
`Entities`.
    
    Previously, `Entities` was too much of a dumping ground for lots of methods 
related to entities, and has become huge! We've previously moved some stuff 
from it, such as `sanitize` to `Sanitizer`. This PR continues that trend.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/aledsage/brooklyn-server separate-dumper-class

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/brooklyn-server/pull/864.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #864
    
----
commit dca846e9fb197375d0485dacdcabfa483de4a93d
Author: Aled Sage <[email protected]>
Date:   2017-10-09T08:51:46Z

    Move Entities.dumpInfo() methods to Dumper class

commit 42c38686304462f2049edc858b5a571991e4abd3
Author: Aled Sage <[email protected]>
Date:   2017-10-09T08:55:33Z

    Dumper: change methods to package-private
    
    And fix deprecated code usage

commit f69c695a298729b3b4551139a2f51956d3c1c7c6
Author: Aled Sage <[email protected]>
Date:   2017-10-17T17:33:05Z

    Use `Dumper.dumpInfo` instead of `Entities.dumpInfo`

----


---

Reply via email to