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

2015-11-22 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/493#discussion_r45574763 --- Diff: components/org.apache.stratos.common/src/main/java/org/apache/stratos/common/statistics/publisher/ThriftStatisticsPublisher.java --- @@ -61,54

[GitHub] stratos pull request: implementing STRATOS-1628

2015-11-21 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/492#discussion_r45546039 --- Diff: components/org.apache.stratos.mock.iaas.client/src/main/java/org/apache/stratos/mock/iaas/client/MockIaasApiClient.java --- @@ -48,6 +49,20

[GitHub] stratos pull request: Topology init

2015-11-09 Thread lahirus
Github user lahirus commented on the pull request: https://github.com/apache/stratos/pull/487#issuecomment-155049020 Thanks for the info Akila. --- 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: Topology init

2015-11-08 Thread lahirus
Github user lahirus commented on the pull request: https://github.com/apache/stratos/pull/487#issuecomment-154924258 Hi Imesh, Do we plan to remove existing, periodically published complete topology? Thanks. --- If your project is set up for it, you can reply to

[GitHub] stratos pull request: GSoC - Curve Fitting

2015-08-12 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/382#discussion_r36847922 --- Diff: components/org.apache.stratos.messaging/src/main/java/org/apache/stratos/messaging/message/processor/health/stat

[GitHub] stratos pull request: GSoC - Curve Fitting

2015-08-11 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/382#discussion_r36823119 --- Diff: components/org.apache.stratos.messaging/src/main/java/org/apache/stratos/messaging/event/health/stat/MemberCurveFinderOfLoadAverageEvent.java

[GitHub] stratos pull request: Merging master with data-publisher-integrati...

2015-08-11 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/426#discussion_r36764561 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/monitor/component/GroupMonitor.java --- @@ -793,13 +780,13

[GitHub] stratos pull request: Merging master with data-publisher-integrati...

2015-08-11 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/426#discussion_r36763910 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/context/partition/ClusterLevelPartitionContext.java

[GitHub] stratos pull request: Merging master with data-publisher-integrati...

2015-08-11 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/426#discussion_r36763683 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/context/partition/ClusterLevelPartitionContext.java

[GitHub] stratos pull request: Merging master with data-publisher-integrati...

2015-08-11 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/426#discussion_r36763617 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/context/cluster/ClusterInstanceContext.java --- @@ -40,22

[GitHub] stratos pull request: Merging master with data-publisher-integrati...

2015-08-11 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/426#discussion_r36763523 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/context/InstanceContext.java --- @@ -29,34 +29,19

[GitHub] stratos pull request: Merging master with data-publisher-integrati...

2015-08-11 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/426#discussion_r36763402 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/applications/parser/DefaultApplicationParser.java

[GitHub] stratos pull request: Merging master with data-publisher-integrati...

2015-08-11 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/426#discussion_r36763342 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/applications/parser/DefaultApplicationParser.java

[GitHub] stratos pull request: Merging master with data-publisher-integrati...

2015-08-11 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/426#discussion_r36763310 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/applications/parser/DefaultApplicationParser.java

[GitHub] stratos pull request: Remove UUID and tenantId from CartridgeBean

2015-08-06 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/418#discussion_r36434651 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41.java --- @@ -302,17 +302,17

[GitHub] stratos pull request: Data publisher integration

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/407#discussion_r35694089 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/client/AutoscalerCloudControllerClient.java --- @@ -228,7

[GitHub] stratos pull request: Adding database model files

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/408#discussion_r35693372 --- Diff: components/org.apache.stratos.manager/META-INF/persistence --- @@ -0,0 +1,20 @@ + --- End diff -- Add licence header --- If your

[GitHub] stratos pull request: Adding database model files

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/408#discussion_r35693429 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/Persistence/PersistenceManager.java --- @@ -0,0 +1,259

[GitHub] stratos pull request: Adding database model files

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/408#discussion_r35693178 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/Persistence/PersistenceManager.java --- @@ -0,0 +1,259

[GitHub] stratos pull request: Adding database model files

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/408#discussion_r35693044 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/Persistence/PersistenceManager.java --- @@ -0,0 +1,259

[GitHub] stratos pull request: Adding database model files

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/408#discussion_r35692977 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/Persistence/PersistenceManager.java --- @@ -0,0 +1,259

[GitHub] stratos pull request: Adding database model files

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/408#discussion_r35692876 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/Persistence/PersistenceManager.java --- @@ -0,0 +1,259

[GitHub] stratos pull request: Adding database model files

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/408#discussion_r35692814 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/Persistence/PersistenceManager.java --- @@ -0,0 +1,259

[GitHub] stratos pull request: Adding database model files

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/408#discussion_r35692752 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/Persistence/PersistenceManager.java --- @@ -0,0 +1,259

[GitHub] stratos pull request: Adding database model files

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/408#discussion_r35692726 --- Diff: components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/Persistence/PersistenceManager.java --- @@ -0,0 +1,259

[GitHub] stratos pull request: Fixing formatting issues and error messages

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/405#discussion_r35650668 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -3127,22 +3128,14

[GitHub] stratos pull request: Fixing formatting issues and error messages

2015-07-28 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/405#discussion_r35650531 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -3127,22 +3128,14

[GitHub] stratos pull request: Data publisher integration

2015-07-22 Thread lahirus
Github user lahirus commented on the pull request: https://github.com/apache/stratos/pull/403#issuecomment-123775024 Is it possible to avoid jar file?( DAS-Artifacts/sparkudf-1.0-SNAPSHOT.jar) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] stratos pull request: Data publisher integration

2015-07-22 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/403#discussion_r35230789 --- Diff: components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/util/CloudControllerConstants.java

[GitHub] stratos pull request: Data publisher integration

2015-07-22 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/403#discussion_r35230668 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/rule/RuleTasksDelegator.java --- @@ -44,347 +43,350

[GitHub] stratos pull request: GSoC - Curve Fitting

2015-07-22 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/382#discussion_r35230388 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/rule/RuleTasksDelegator.java --- @@ -357,6 +378,12 @@ public

[GitHub] stratos pull request: GSoC - Curve Fitting

2015-07-22 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/382#discussion_r35230422 --- Diff: components/org.apache.stratos.messaging/src/main/java/org/apache/stratos/messaging/event/health/stat/CurveFinderOfLoadAverageEvent.java

[GitHub] stratos pull request: Change group validation with a local variabl...

2015-06-20 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/373#discussion_r32883671 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -1622,6 +1626,8

[GitHub] stratos pull request: Fixed update application command in cli

2015-05-20 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/354#discussion_r30730518 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/util/AutoscalerUtil.java --- @@ -773,19 +773,34 @@ public

[GitHub] stratos pull request: Fixed issues found when removing a non exist...

2015-05-20 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/353#discussion_r30725808 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41.java --- @@ -426,10 +427,21

[GitHub] stratos pull request: Fixed issues found when removing a non exist...

2015-05-20 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/353#discussion_r30707196 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -252,10 +251,12

[GitHub] stratos pull request: Fixed deactivating non-exist tenant in cli

2015-05-18 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/329#discussion_r30491310 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -3170,7 +3170,27

[GitHub] stratos pull request: Introducing custom exceptions for applicatio...

2015-05-17 Thread lahirus
Github user lahirus commented on the pull request: https://github.com/apache/stratos/pull/327#issuecomment-102925171 Thanks Pubudu. --- 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

[GitHub] stratos pull request: Introducing custom exceptions for applicatio...

2015-05-17 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/327#discussion_r30473872 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -699,7 +699,7

[GitHub] stratos pull request: Introducing custom exceptions for applicatio...

2015-05-17 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/327#discussion_r30473853 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41.java --- @@ -817,12 +817,18

[GitHub] stratos pull request: Adding api method for updating cartridge gro...

2015-05-13 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/311#discussion_r30249647 --- Diff: components/org.apache.stratos.autoscaler/src/main/java/org/apache/stratos/autoscaler/registry/RegistryManager.java --- @@ -348,6 +349,28

[GitHub] stratos pull request: Changing the CLI commands and Rest API

2015-05-12 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/303#discussion_r30113131 --- Diff: components/org.apache.stratos.cli/src/main/java/org/apache/stratos/cli/commands/RemoveDomainMappingCommand.java --- @@ -0,0 +1,125

[GitHub] stratos pull request: Moving non api methods

2015-05-09 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/298#discussion_r29998288 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -2932,7 +2946,7

[GitHub] stratos pull request: Moving non api methods

2015-05-09 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/298#discussion_r29998276 --- 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-09 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/298#discussion_r29998268 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -2781,7 +2782,7

[GitHub] stratos pull request: Moving non api methods

2015-05-09 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/298#discussion_r29998263 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -2751,15 +2751,16

[GitHub] stratos pull request: Service group validation

2015-05-06 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/296#discussion_r29763506 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -980,50 +981,41