[GitHub] stratos pull request: Closing STRATOS-1641: Validate member status...

2015-12-07 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/501#discussion_r46919246 --- Diff: components/org.apache.stratos.rest.endpoint/src/test/java/org/apache/stratos/rest/endpoint/test/UndeployApplicationTest.java --- @@ -0,0 +1,212

[GitHub] stratos pull request: Fixing STRATOS-1632 and updating cep and das...

2015-11-30 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/496#discussion_r46130001 --- Diff: components/org.apache.stratos.common/src/test/java/org/apache/stratos/common/test/ThriftStatisticsPublisherTest.java --- @@ -0,0 +1,60

[GitHub] stratos pull request: Fixing STRATOS-1632 and updating cep and das...

2015-11-30 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/496#discussion_r46129874 --- Diff: components/org.apache.stratos.common/src/test/java/org/apache/stratos/common/test/ThriftStatisticsPublisherTest.java --- @@ -0,0 +1,60

[GitHub] stratos pull request: Fixing STRATOS-1632 and updating cep and das...

2015-11-29 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/496#discussion_r46110104 --- Diff: extensions/cep/modules/distribution/src/assembly/bin.xml --- @@ -56,4 +56,13 @@ 644

[GitHub] stratos pull request: Fixing STRATOS-1632 and updating cep and das...

2015-11-29 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/496#discussion_r46110075 --- Diff: extensions/cep/modules/distribution/pom.xml --- @@ -63,4 +63,14

[GitHub] stratos pull request: Fixing STRATOS-1632 and updating cep and das...

2015-11-29 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/496#discussion_r46110016 --- Diff: components/org.apache.stratos.common/src/main/java/org/apache/stratos/common/statistics/publisher/ThriftStatisticsPublisher.java --- @@ -85,11

[GitHub] stratos pull request: Fixing STRATOS-1632 and updating cep and das...

2015-11-29 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/496#discussion_r46109995 --- Diff: components/org.apache.stratos.common/src/main/java/org/apache/stratos/common/statistics/publisher/ThriftStatisticsPublisher.java --- @@ -85,11

[GitHub] stratos pull request: Fixing multiple publisher objects created fo...

2015-11-23 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/493#discussion_r45602771 --- Diff: products/stratos/modules/distribution/src/main/conf/drools/mincheck.drl --- @@ -83,9 +81,8 @@ dialect "mvel"

[GitHub] stratos pull request: Fixing multiple publisher objects created fo...

2015-11-23 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/493#discussion_r45602412 --- Diff: components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/statistics/publisher/DASMemberStatusPublisher.java

[GitHub] stratos pull request: Fixing multiple publisher objects created fo...

2015-11-22 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/493#discussion_r45575684 --- Diff: products/stratos/modules/distribution/src/main/conf/drools/mincheck.drl --- @@ -83,9 +81,8 @@ dialect "mvel"

[GitHub] stratos pull request: Fixing multiple publisher objects created fo...

2015-11-22 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/493#discussion_r45575676 --- Diff: components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/statistics/publisher/DASMemberStatusPublisher.java

[GitHub] stratos pull request: Making signup button available only when app...

2015-09-14 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/464#discussion_r39477594 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/util/converter/ObjectConverter.java --- @@ -1147,15

[GitHub] stratos pull request: Thrift Client Configuration Modification

2015-08-20 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/443#discussion_r37607948 --- Diff: components/org.apache.stratos.common/src/main/java/org/apache/stratos/common/statistics/publisher/wso2/cep/WSO2CEPHealthStatisticsPublisher.java

[GitHub] stratos pull request: Thrift Client Configuration Modification

2015-08-20 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/443#discussion_r37607952 --- Diff: components/org.apache.stratos.common/src/main/java/org/apache/stratos/common/statistics/publisher/wso2/cep/WSO2CEPInFlightRequestPublisher.java

[GitHub] stratos pull request: Google compute engine load balancing support

2015-08-16 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/433#discussion_r37158810 --- Diff: extensions/load-balancer/gce-extension/src/main/java/org/apache/stratos/gce/extension/util/GCEOperations.java --- @@ -0,0 +1,589

[GitHub] stratos pull request: Google compute engine load balancing support

2015-08-16 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/433#discussion_r37158573 --- Diff: extensions/load-balancer/gce-extension/src/main/java/org/apache/stratos/gce/extension/util/GCEOperations.java --- @@ -0,0 +1,589

[GitHub] stratos pull request: Google compute engine load balancing support...

2015-08-10 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/419#discussion_r36712387 --- Diff: extensions/load-balancer/gce-extension/src/main/java/org/apache/stratos/gce/extension/util/GCEOperations.java --- @@ -0,0 +1,588

[GitHub] stratos pull request: Google compute engine load balancing support...

2015-08-10 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/419#issuecomment-129707792 Hi Asanka, Please use the log.error(object,throwable) method when you are log the error logs with the exception catch block. This is the standard we have

[GitHub] stratos pull request: Google compute engine load balancing support...

2015-08-10 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/419#discussion_r36712101 --- Diff: extensions/load-balancer/gce-extension/src/main/java/org/apache/stratos/gce/extension/util/GCEOperations.java --- @@ -0,0 +1,588

[GitHub] stratos pull request: Google compute engine load balancing support...

2015-08-10 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/419#discussion_r36712081 --- Diff: extensions/load-balancer/gce-extension/src/main/java/org/apache/stratos/gce/extension/util/GCEOperations.java --- @@ -0,0 +1,588

[GitHub] stratos pull request: fixing issues in GCEIaas class and gce confi...

2015-06-03 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/365#discussion_r31694369 --- Diff: tools/config-scripts/gce/config.sh --- @@ -90,7 +91,10 @@ if [[ $answer = y ]] ; then ARGS=("-n${NODEID}" "-d${DOMAIN}&

[GitHub] stratos pull request: Adding Openstack init,config scripts and fix...

2015-06-03 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/366#discussion_r31617873 --- Diff: tools/config-scripts/openstack/config.sh --- @@ -0,0 +1,104 @@ +#!/bin/bash

[GitHub] stratos pull request: Moving non api methods

2015-05-10 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/298#discussion_r30002527 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -3179,7 +3201,9

[GitHub] stratos pull request: Moving non api methods

2015-05-10 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/298#discussion_r30002518 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -3123,7 +3141,9

[GitHub] stratos pull request: Moving non api methods

2015-05-10 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/298#discussion_r30002496 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -3097,7 +3113,9

[GitHub] stratos pull request: Moving non api methods

2015-05-10 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/298#discussion_r30002483 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -2836,52 +2838,58

[GitHub] stratos pull request: Moving non api methods

2015-05-10 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/298#discussion_r30002470 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -2836,52 +2838,58

[GitHub] stratos pull request: Service group validation

2015-05-06 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/296#discussion_r29754432 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41.java --- @@ -453,6 +453,15 @@ public

[GitHub] stratos pull request: Adding properties to add payload parameters ...

2015-04-27 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/289#discussion_r29138259 --- Diff: components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/services/impl/CloudControllerServiceImpl.java

[GitHub] stratos pull request: Adding network partitions for CLI

2015-01-29 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/198#discussion_r23768119 --- Diff: components/org.apache.stratos.cli/src/main/java/org/apache/stratos/cli/RestCommandLineService.java --- @@ -1130,4 +1134,57 @@ private void

[GitHub] stratos pull request: Adding network partitions for CLI

2015-01-29 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/198#discussion_r23767029 --- Diff: components/org.apache.stratos.cli/src/main/java/org/apache/stratos/cli/commands/AddNetworkPartitionCommand.java --- @@ -0,0 +1,174

[GitHub] stratos pull request: Adding some configuration changes related to...

2015-01-18 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/131#issuecomment-70449314 This is already merge with latest code base.Hence closing the PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] stratos pull request: Adding some configuration changes related to...

2015-01-18 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/131 --- 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

[GitHub] stratos pull request: Adding executor service for Startos

2015-01-18 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/128 --- 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

[GitHub] stratos pull request: Adding executor service for Startos

2015-01-18 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/128#issuecomment-70449229 This is already merge with the latest code base.Hence closing the PR --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] stratos pull request: Rest api StratosApiV41Utils.java Refactor

2015-01-18 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/157#issuecomment-70449161 This is already merged with the code base.Hence closing the pull request. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] stratos pull request: Rest api StratosApiV41Utils.java Refactor

2015-01-18 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/157 --- 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

[GitHub] stratos pull request: Re-base and merge the PR#157

2015-01-18 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/158#issuecomment-70449055 This is already merged in the code base.Hence closing the PR --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] stratos pull request: Re-base and merge the PR#157

2015-01-18 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/158 --- 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

[GitHub] stratos pull request: Meta Data Service - Code Refactors

2015-01-18 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/116#issuecomment-70448716 Thanks Imesh. This is a improvement. We will merge this improvement after the Alpha release.Hence closing the PR Thanks, Gayan --- If your project is set up

[GitHub] stratos pull request: Meta Data Service - Code Refactors

2015-01-18 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/116 --- 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

[GitHub] stratos pull request: Refactor code with reformatting

2015-01-12 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/110#issuecomment-69702343 Those changes are already merged.Hence closing the PR --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] stratos pull request: Refactor code with reformatting

2015-01-12 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/110 --- 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

[GitHub] stratos pull request: Fixes for error in publishing artifact updat...

2015-01-12 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/91#issuecomment-69698234 This is merged with PR#87.Hence closing this PR. --- 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

[GitHub] stratos pull request: Fixes for error in publishing artifact updat...

2015-01-12 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/91 --- 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

[GitHub] stratos pull request: Re-factoring of the meta data service

2015-01-12 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/83 --- 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

[GitHub] stratos pull request: Re-factoring of the meta data service

2015-01-12 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/83#issuecomment-69698161 Those changes are merged with PR#110.Hence closing this PR --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] stratos pull request: Modify configuration name conventions

2015-01-12 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/56 --- 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

[GitHub] stratos pull request: Modify configuration name conventions

2015-01-12 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/56#issuecomment-69698110 This is merged with #110. Hence closing the PR --- 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

[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: Refactor messaging util class

2014-12-21 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/162 --- 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

[GitHub] stratos pull request: Refactor messaging util class

2014-12-21 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/162#issuecomment-67807555 Thanks Imesh for merging --- 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

[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

[GitHub] stratos pull request: Meta data service improvement with code refa...

2014-12-21 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/119#issuecomment-67806810 Thanks. This is cover with PR#162 --- 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

[GitHub] stratos pull request: Refactor messaging util class

2014-12-21 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/162 Refactor messaging util class This PR will update messaging Util class with constants. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] stratos pull request: Add the context aware error messages

2014-12-20 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/159#issuecomment-67738841 Thanks Imesh for merging this PR --- 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

[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

[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

[GitHub] stratos pull request: Add the context aware error messages

2014-12-20 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/159 Add the context aware error messages Add the context aware error messages. Please review and merge You can merge this pull request into a Git repository by running: $ git pull

[GitHub] stratos pull request: Re-base and merge the PR#157

2014-12-19 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/158 Re-base and merge the PR#157 This PR contains the followings. Update the class with String format for string generation Update error handling in rest API and meta data service

[GitHub] stratos pull request: Add the rest api operation for list cartridg...

2014-12-19 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/136#issuecomment-67627905 Thanks Nuwan. This is changed with PR#157 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] stratos pull request: Re-factoring of the meta data service

2014-12-19 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/84#discussion_r2218 --- Diff: components/org.apache.stratos.metadataservice/src/main/java/org/apache/stratos/metadataservice/registry/CarbonRegistry.java --- @@ -31,44 +31,121

[GitHub] stratos pull request: Re-factoring of the meta data service

2014-12-19 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/84#discussion_r22099924 --- Diff: components/org.apache.stratos.metadataservice/src/main/java/org/apache/stratos/metadataservice/registry/GRegRegistry.java --- @@ -18,91 +18,199

[GitHub] stratos pull request: Re-factoring of the meta data service

2014-12-19 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/84#discussion_r22099926 --- Diff: components/org.apache.stratos.metadataservice/src/main/java/org/apache/stratos/metadataservice/registry/GRegRegistry.java --- @@ -18,91 +18,199

[GitHub] stratos pull request: Re-factoring of the meta data service

2014-12-19 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/84#discussion_r22098774 --- Diff: components/org.apache.stratos.metadataservice/src/main/java/org/apache/stratos/metadataservice/services/MetaDataAdmin.java --- @@ -24,36 +43,67

[GitHub] stratos pull request: Meta data service improvement with code refa...

2014-12-19 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/119#discussion_r22098577 --- Diff: components/org.apache.stratos.metadataservice/src/main/java/org/apache/stratos/metadataservice/registry/CarbonRegistry.java --- @@ -112,7 +116,7

[GitHub] stratos pull request: Meta data service improvement with code refa...

2014-12-19 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/119#discussion_r22098538 --- Diff: components/org.apache.stratos.messaging/src/main/java/org/apache/stratos/messaging/util/Util.java --- @@ -94,12 +101,17 @@ else if (isNumber(array

[GitHub] stratos pull request: Add the rest api operation for list cartridg...

2014-12-19 Thread swgkg
Github user swgkg commented on a diff in the pull request: https://github.com/apache/stratos/pull/136#discussion_r22096345 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -725,6 +725,91

[GitHub] stratos pull request: Update the categary of cartridge as a mandat...

2014-12-19 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/145#issuecomment-67613970 Thanks Ramith. Yeah StringUtils.isBlank() is more suitable there. It will cover with PR#157 --- If your project is set up for it, you can reply to this email

[GitHub] stratos pull request: Rest api StratosApiV41Utils.java Refactor

2014-12-19 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/157 Rest api StratosApiV41Utils.java Refactor This PR contains the followings. 1. Add the String.format for error message handling. 2. Add method doc comment. You can merge this pull

[GitHub] stratos pull request: Add the rest api operation for list cartridg...

2014-12-18 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/136#issuecomment-67485237 Thanks Kasun for pointing out this. Yeah it is better to have whole exception than only the message. I added PR#154 for this. Thanks, Gayan

[GitHub] stratos pull request: Make group monitor as a schedule service

2014-12-15 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/154 Make group monitor as a schedule service This PR will Make group monitor as a schedule service You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] stratos pull request: Make group monitor as a schedule service

2014-12-15 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/151#issuecomment-67123402 Thanks Imesh. I will re-base the branch and create a new PR --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] stratos pull request: Make group monitor as a schedule service

2014-12-15 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/151 --- 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

[GitHub] stratos pull request: Make group monitor as a schedule service

2014-12-13 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/151 Make group monitor as a schedule service This PR contains: Make group monitor as a schedule service. Add a map to maintain the child scaling events with in the group monitor Please

[GitHub] stratos pull request: Update the info message with application ID

2014-12-13 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/148#issuecomment-66882630 Thanks Imesh. Hence closing the PR --- 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

[GitHub] stratos pull request: Update the info message with application ID

2014-12-13 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/148 --- 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

[GitHub] stratos pull request: Update the info message with application ID

2014-12-11 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/148 Update the info message with application ID Update the info message with application ID Please review and merge You can merge this pull request into a Git repository by running: $ git

[GitHub] stratos pull request: Update the categary of cartridge as a mandat...

2014-12-11 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/145 --- 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

[GitHub] stratos pull request: Update the categary of cartridge as a mandat...

2014-12-11 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/145#issuecomment-66599365 This PR is merged hence closing the PR --- 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

[GitHub] stratos pull request: Update the categary of cartridge as a mandat...

2014-12-09 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/145 Update the categary of cartridge as a mandatory value This pr will Update the categary of cartridge as a mandatory value Please review and merge You can merge this pull request into a Git

[GitHub] stratos pull request: Update cleanup clustor with the instance id

2014-12-09 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/143 Update cleanup clustor with the instance id Added the instance id condition for the clean up cluster Please review and merge You can merge this pull request into a Git repository by running

[GitHub] stratos pull request: Adding instance id for health stats publishe...

2014-12-09 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/140#issuecomment-66277245 This is already merge , hence closing the pull request commit ID :adf60a8608d27879ce9630cc8531bfc953f21bc9 --- If your project is set up for it, you can reply to

[GitHub] stratos pull request: Adding instance id for health stats publishe...

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

[GitHub] stratos pull request: Adding instance id for health stats publishe...

2014-12-08 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/140 Adding instance id for health stats publisher in cartridge agent This PR contains: Adding the instance id for health stats publisher in cartridge agent Adding executor service for haproxy

[GitHub] stratos pull request: Add the rest api operation for list cartridg...

2014-12-04 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/136 Add the rest api operation for list cartridges based on the cartridge provider Related JIRA: STRATOS-1023 Please review and merge You can merge this pull request into a Git repository by

[GitHub] stratos pull request: Update the python agent event publisher with...

2014-12-04 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/135 Update the python agent event publisher with instance_id Please review and merge You can merge this pull request into a Git repository by running: $ git pull https://github.com/gayangunarathne

[GitHub] stratos pull request: Update the ConfUtil to read multiple configu...

2014-12-04 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/133 --- 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

[GitHub] stratos pull request: Update the ConfUtil to read multiple configu...

2014-12-04 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/133#issuecomment-65633972 This PR is merged hence closing the PR --- 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

[GitHub] stratos pull request: Adding the support to read multiple configur...

2014-12-03 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/133 Adding the support to read multiple configuration files Adding the support to read multiple configuration files Please review and merge You can merge this pull request into a Git repository

[GitHub] stratos pull request: Re-base and merge changes of PR 131

2014-12-03 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/132 Re-base and merge changes of PR 131 Re-base and merge changes of PR 131 Please review and merge You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] stratos pull request: Adding some configuration changes related to...

2014-12-03 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/131 Adding some configuration changes related to the executor service This PR contains some configuration changes related to the executor service. Please review and merge You can merge this

[GitHub] stratos pull request: Adding instance id for health stats publishe...

2014-12-02 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/130 Adding instance id for health stats publisher in cartridge agent Adding instance id for health stats publisher in cartridge agent Related jira : STRATOS-1019 Please review and merge

[GitHub] stratos pull request: Merge the master with the changes of PR #128

2014-12-02 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/129 Merge the master with the changes of PR #128 This PR contains the following changes: Remove unnecessary threads in messaging model Adding executor service for threads Related

[GitHub] stratos pull request: Adding executor service for Startos

2014-12-01 Thread swgkg
GitHub user swgkg opened a pull request: https://github.com/apache/stratos/pull/128 Adding executor service for Startos This PR contains the following changes: Remove unnecessary threads in messaging model Adding executor service for threads Related JIRA -https

[GitHub] stratos pull request: Merge the master with the changes of PR #121

2014-11-30 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/125#issuecomment-65020107 Changes merge with #127 , hence closing the PR --- 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

[GitHub] stratos pull request: Introduce the enum for messaging topics

2014-11-30 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/121 --- 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

[GitHub] stratos pull request: Introduce the enum for messaging topics

2014-11-30 Thread swgkg
Github user swgkg commented on the pull request: https://github.com/apache/stratos/pull/121#issuecomment-65020126 Changes merge with #127 , hence closing the PR --- 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

[GitHub] stratos pull request: Merge the master with the changes of PR #121

2014-11-30 Thread swgkg
Github user swgkg closed the pull request at: https://github.com/apache/stratos/pull/125 --- 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

  1   2   >