[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-12-21 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/50#issuecomment-67807297 Thanks Johann for the comments. Added the PR#159 with the context aware error messages. --- If your project is set up for it, you can reply to this email and have

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-12-21 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/50#discussion_r22155958 --- Diff: components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/internal/CloudControllerDSComponent.java --- @@ -44,

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-12-20 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/50#discussion_r22141443 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/listener/InstanceStatusListener.java --- @@ -26,71 +31,108 @@ imp

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-12-20 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/50#issuecomment-67734371 Thanks Johann for the comments. Added the PR#159 with the context aware error messages. --- If your project is set up for it, you can reply to this email and have

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-12-20 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/50#issuecomment-67734285 Thanks Johann for point out this. Added the PR#159 with the context aware error messages. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-12-19 Thread johannnallathamby
Github user johannnallathamby commented on a diff in the pull request: https://github.com/apache/stratos/pull/50#discussion_r22122620 --- Diff: components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/topic/instance/status/InstanceStatusEventMess

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-12-19 Thread johannnallathamby
Github user johannnallathamby commented on a diff in the pull request: https://github.com/apache/stratos/pull/50#discussion_r22122521 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/publisher/TenantEventPublisher.java --- @@ -38,78 +38,83 @

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-12-19 Thread johannnallathamby
Github user johannnallathamby commented on a diff in the pull request: https://github.com/apache/stratos/pull/50#discussion_r22122471 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/listener/InstanceStatusListener.java --- @@ -26,71 +31,108

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-12-19 Thread johannnallathamby
Github user johannnallathamby commented on a diff in the pull request: https://github.com/apache/stratos/pull/50#discussion_r22122404 --- Diff: components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/internal/CloudControllerDSComponent.java ---

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-09-24 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/50#issuecomment-56657661 This is already merged and add the new pull request (https://github.com/apache/stratos/pull/58) for STRATOS-820. Hence closed the PR --- If your project is set up for it,

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-09-24 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/50 --- 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 enabl

Re: [GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-09-23 Thread Gayan Gunarathne
Sure. I will send the pull request for this. On Wed, Sep 24, 2014 at 10:49 AM, nirmal070125 wrote: > Github user nirmal070125 commented on the pull request: > > https://github.com/apache/stratos/pull/50#issuecomment-56625991 > > I solved merge conflicts and integrated this PR. Thanks Gay

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-09-23 Thread nirmal070125
Github user nirmal070125 commented on the pull request: https://github.com/apache/stratos/pull/50#issuecomment-56625991 I solved merge conflicts and integrated this PR. Thanks Gayan. However, later found that this PR has introduced a regression https://issues.apache.org/jira/b

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-09-23 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/50 Messaging model changes and MQTT integration Please review and merge You can merge this pull request into a Git repository by running: $ git pull https://github.com/gayangunarathne/stratos msgtop

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-09-23 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/47 --- 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 enabl

[GitHub] stratos pull request: Messaging model changes and MQTT integration

2014-09-22 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/47 Messaging model changes and MQTT integration Please review and merge You can merge this pull request into a Git repository by running: $ git pull https://github.com/gayangunarathne/stratos msgtop