stratos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (STRATOS-1632) DataPublisher objects are created for nodes which has statsPublisherEnabled set to false in thrift-client-config.xml
Date Mon, 30 Nov 2015 05:39:10 GMT

    [ https://issues.apache.org/jira/browse/STRATOS-1632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15031337#comment-15031337
] 

ASF GitHub Bot commented on STRATOS-1632:
-----------------------------------------

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 +85,13 @@ private void init() {
             ArrayList<ReceiverGroup> receiverGroups = new ArrayList<ReceiverGroup>();
     
             for (ThriftClientInfo thriftClientInfo : thriftClientInfoList) {
    -            ArrayList<DataPublisherHolder> dataPublisherHolders = new ArrayList<DataPublisherHolder>();
    -            DataPublisherHolder aNode = new DataPublisherHolder(null, buildUrl(thriftClientInfo),
thriftClientInfo.getUsername(), thriftClientInfo.getPassword());
    -            dataPublisherHolders.add(aNode);
    -            ReceiverGroup group = new ReceiverGroup(dataPublisherHolders);
    -            receiverGroups.add(group);
    +            if (thriftClientInfo.isStatsPublisherEnabled()) {
    +                ArrayList<DataPublisherHolder> dataPublisherHolders = new ArrayList<DataPublisherHolder>();
    +                DataPublisherHolder aNode = new DataPublisherHolder(null, buildUrl(thriftClientInfo),
thriftClientInfo.getUsername(), thriftClientInfo.getPassword());
    +                dataPublisherHolders.add(aNode);
    +                ReceiverGroup group = new ReceiverGroup(dataPublisherHolders);
    +                receiverGroups.add(group);
    +            }
             }
             return receiverGroups;
     
    --- End diff --
    
    Also it is better to have some debug logs on this flow.


> DataPublisher objects are created for nodes which has statsPublisherEnabled set to false
in thrift-client-config.xml
> --------------------------------------------------------------------------------------------------------------------
>
>                 Key: STRATOS-1632
>                 URL: https://issues.apache.org/jira/browse/STRATOS-1632
>             Project: Stratos
>          Issue Type: Bug
>    Affects Versions: 4.1.4, 4.1.5
>            Reporter: Thanuja Uruththirakodeeswaran
>
>  When we have enabled stats publishing for one node and disabled for another node in
thrift-client-config.xml file, Stratos is trying to publish data to both nodes. This is because
when creating load balancing type data publisher, it is creating DataPublisher for ThriftClientInfo
object which has false value set to statsPublisherEnabled attribute.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message