[GitHub] incubator-brooklyn pull request: Mirror remote effectors

[neykov <gi...@git.apache.org>]

GitHub user neykov opened a pull request:

    https://github.com/apache/incubator-brooklyn/pull/181

    Mirror remote effectors

    Parameter have Object types resulting in transparently passing the incoming data to the remote effector.
    Effectors' return is treated as plain string.
    
    Needs tests and possibly converting the return type to object with additional logic when the response is empty (and other cases?)

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

    $ git pull https://github.com/neykov/incubator-brooklyn mirror-effectors

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

    https://github.com/apache/incubator-brooklyn/pull/181.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 #181
    
----
commit 0b269ba33816a34006827c9eb49c7a80f0e2a59c
Author: Svetoslav Neykov <sv...@cloudsoftcorp.com>
Date:   2014-09-19T14:52:18Z

    BrooklynEntityMirror: Mirror effectors of remote entity
    
    Parameter have Object types resulting in transparently passing the incoming data to the remote effector.
    Effectors' return is treated as plain string.

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-brooklyn pull request: Mirror remote effectors

[asfgit <gi...@git.apache.org>]

Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-brooklyn/pull/181


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-brooklyn pull request: Mirror remote effectors

[ahgittin <gi...@git.apache.org>]

Github user ahgittin commented on the pull request:

    https://github.com/apache/incubator-brooklyn/pull/181#issuecomment-56388633
  
    first pass looks good.  merging to prevent conflicts, will do more reviews after.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---