[
https://issues.apache.org/jira/browse/STORM-885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15004357#comment-15004357
]
ASF GitHub Bot commented on STORM-885:
--------------------------------------
Github user ptgoetz commented on the pull request:
https://github.com/apache/storm/pull/838#issuecomment-156498367
I'm +1 for adding this functionality, but I would say that vote is
non-binding since this is new code/functionality I'm not that familiar with
yet. The code looks good to me, but I would defer to those more familiar with
it to determine if it's ready.
I would like to see more documentation. Specifically:
* Rationale
* Benefits
* Configuration/Usage guidelines
I'm okay with handling that as a separate JIRA, but I think it would be
needed before we include this in a release.
> 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)
