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

ASF GitHub Bot commented on STORM-885:
--------------------------------------

Github user erikdw commented on a diff in the pull request:

    https://github.com/apache/storm/pull/838#discussion_r45117066
  
    --- Diff: storm-core/src/jvm/backtype/storm/Config.java ---
    @@ -778,6 +792,47 @@
         public static final String UI_HTTPS_NEED_CLIENT_AUTH = 
"ui.https.need.client.auth";
     
         /**
    +     * The host that Pacemaker is running on.
    +     */
    +    @isString
    +    public static final String PACEMAKER_HOST = "pacemaker.host";
    +
    +    /**
    +     * The port Pacemaker should run on. Clients should
    +     * connect to this port to submit or read heartbeats.
    +     */
    +    @isNumber
    +    @isPositiveNumber
    +    public static final String PACEMAKER_PORT = "pacemaker.port";
    +
    +    /**
    +     * The maximum number of threads that should be used by the Pacemaker.
    +     * When Pacemaker gets loaded it will spawn new threads, up to 
    +     * this many total, to handle the load.
    +     */
    +    @isNumber
    +    @isPositiveNumber
    +    public static final String PACEMAKER_MAX_THREADS = 
"pacemaker.max.threads";
    +
    +    /**
    +     * This parameter is used by the storm-deploy project to configure the
    +     * jvm options for the nimbus daemon.
    --- End diff --
    
    I suppose @knusbaum means: you have a cargo culted string in your comment:  
this config is for the *pacemaker* daemon, **not** the *nimbus* daemon.


> Heartbeat Server (Pacemaker)
> ----------------------------
>
>                 Key: STORM-885
>                 URL: https://issues.apache.org/jira/browse/STORM-885
>             Project: Apache Storm
>          Issue Type: Improvement
>          Components: storm-core
>            Reporter: Robert Joseph Evans
>            Assignee: Kyle Nusbaum
>
> Large highly connected topologies and large clusters write a lot of data into 
> ZooKeeper.  The heartbeats, that make up the majority of this data, do not 
> need to be persisted to disk.  Pacemaker is intended to be a secure 
> replacement for storing the heartbeats without changing anything within the 
> heartbeats.  In the future as more metrics are added in, we may want to look 
> into switching it over to look more like Heron, where a metrics server is 
> running for each node/topology.  And can be used to aggregate/per-aggregate 
> them in a more scalable manor.



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

Reply via email to