You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by abhishekagarwal87 <gi...@git.apache.org> on 2016/06/10 15:49:35 UTC

[GitHub] storm pull request #1481: STORM-1705: Fix cap-retry bug

GitHub user abhishekagarwal87 opened a pull request:

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

    STORM-1705: Fix cap-retry bug

    

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

    $ git pull https://github.com/abhishekagarwal87/storm cap-retry

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

    https://github.com/apache/storm/pull/1481.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 #1481
    
----
commit a59695e97953dc68b798f01df92c49b6df4b6abd
Author: Abhishek Agarwal <ab...@inmobi.com>
Date:   2016-06-10T15:48:40Z

    STORM-1705: Fix cap-retry bug

----


---
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 #1481: STORM-1705: Fix cap-retry bug

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

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


---
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 #1481: STORM-1705: Fix cap-retry bug

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

    https://github.com/apache/storm/pull/1481
  
    Adding @harshach - do you want #1482 (STORM-1876) in 1.0.2? 


---
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 #1481: STORM-1705: Fix cap-retry bug

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

    https://github.com/apache/storm/pull/1481
  
    +1 Thanks @abhishekagarwal87.
    
    Since this is a one-line change blocking the release, I'm going to merge it in the interest of proceeding with the release. If there are any objections we can cancel the next vote and revert the change.


---
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 #1481: STORM-1705: Fix cap-retry bug

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

    https://github.com/apache/storm/pull/1481
  
    Thinking further on it, STORM-1876 is needed when building from the source. So it should be fine if not included in the release candidate. 
    @ptgoetz  - +1 for merging this now and going ahead with release. 


---
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.
---