You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by knusbaum <gi...@git.apache.org> on 2016/09/29 22:08:05 UTC

[GitHub] storm pull request #1720: STORM-1546: Adding Read and Write Aggregations for...

GitHub user knusbaum opened a pull request:

    https://github.com/apache/storm/pull/1720

    STORM-1546: Adding Read and Write Aggregations for Pacemaker to make it HA compatible (1.x-branch)

    1.x-branch version of Pacemaker-HA

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/knusbaum/incubator-storm Pacemaker-HA-1.x-branch2

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/1720.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1720
    
----
commit 7b34582463857eba8bc08ff8abab338f4e83fbd6
Author: Kyle Nusbaum <ky...@gmail.com>
Date:   2016-09-29T18:58:54Z

    Moving files into old position for merge.

commit dd7ff1e0a0cecfe581c7441f941e3b9bbcc90c9d
Author: Sanket <sc...@untilservice-lm>
Date:   2016-03-04T22:41:02Z

    Porting Pacemaker-HA to community.

commit a6171bd6770676d4d0bdb9c9d05a4785df35b671
Author: Kyle Nusbaum <ky...@gmail.com>
Date:   2016-09-29T20:43:09Z

    Ported successfully.

commit 20180cca1de9b25dc391722a9088e10107a31443
Author: Kyle Nusbaum <ky...@gmail.com>
Date:   2016-09-29T20:45:01Z

    Minor cleanup.

commit 9c5860498a71c04fda1e201283a63ec2c6352062
Author: Kyle Nusbaum <ky...@gmail.com>
Date:   2016-09-29T22:06:16Z

    Making HA backwards-compatible

----


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm issue #1720: STORM-1546: Adding Read and Write Aggregations for Pacema...

Posted by knusbaum <gi...@git.apache.org>.
Github user knusbaum commented on the issue:

    https://github.com/apache/storm/pull/1720
  
    @HeartSaVioR I've synced the java files between the branches so there are almost no differences.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm issue #1720: STORM-1546: Adding Read and Write Aggregations for Pacema...

Posted by HeartSaVioR <gi...@git.apache.org>.
Github user HeartSaVioR commented on the issue:

    https://github.com/apache/storm/pull/1720
  
    Seems like this PR has different diff compared to master (#1710) except Clojure things. This could be OK in short time, but will make us bothering if we need to fix something with Pacemaker.
    
    I'd recommend to sync this with #1710 except pacemaker_state_factory.clj. I'm also OK to replace related clojure files to ported files in 1.x since it doesn't matter.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm issue #1720: STORM-1546: Adding Read and Write Aggregations for Pacema...

Posted by HeartSaVioR <gi...@git.apache.org>.
Github user HeartSaVioR commented on the issue:

    https://github.com/apache/storm/pull/1720
  
    Thanks for the change. +1


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm issue #1720: STORM-1546: Adding Read and Write Aggregations for Pacema...

Posted by redsanket <gi...@git.apache.org>.
Github user redsanket commented on the issue:

    https://github.com/apache/storm/pull/1720
  
    +1


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm issue #1720: STORM-1546: Adding Read and Write Aggregations for Pacema...

Posted by HeartSaVioR <gi...@git.apache.org>.
Github user HeartSaVioR commented on the issue:

    https://github.com/apache/storm/pull/1720
  
    Please add a documentation here too.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm issue #1720: STORM-1546: Adding Read and Write Aggregations for Pacema...

Posted by knusbaum <gi...@git.apache.org>.
Github user knusbaum commented on the issue:

    https://github.com/apache/storm/pull/1720
  
    Also updated the docs.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request #1720: STORM-1546: Adding Read and Write Aggregations for...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/storm/pull/1720


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---