GitHub user andreschembri opened a pull request:
https://github.com/apache/incubator-brooklyn/pull/590
Catalogitemsummaryfix
Not sure if this is the correct way to resolve this issue:
- There are some code that is using this POJO that are providing the
registeredType and others that do not.
- If the registeredType is present and the constructor does not take it ,
the mapping would fail, if registeredType is not present and the constructor
expects the registeredType the number of arguments would not match.
- I ended up adding a new constructor that takes the registeredType, and
leaving the one that does not in place
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/andreschembri/incubator-brooklyn
catalogitemsummaryfix
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-brooklyn/pull/590.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 #590
----
commit 5512dae60c5cf9bc5c1c855e0074d6e4406ca956
Author: Andre <[email protected]>
Date: 2015-04-09T09:51:32Z
added representation of registeredType to CatalogItemSummary
commit 72f040069e566e6a0c6ef0d7440d92c7ea7c8969
Author: Andre <[email protected]>
Date: 2015-04-09T14:47:10Z
Added another constructor for when registeredType is not present
----
---
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.
---
