You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by ptgoetz <gi...@git.apache.org> on 2014/03/26 17:40:44 UTC
[GitHub] incubator-storm pull request: STORM-263: update to carbonite 1.4.0...
GitHub user ptgoetz opened a pull request:
https://github.com/apache/incubator-storm/pull/56
STORM-263: update to carbonite 1.4.0/kryo 2.21
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ptgoetz/incubator-storm STORM-263
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-storm/pull/56.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 #56
----
commit 447fdb0b56757cd8d97d7c684f8a0b3b68639673
Author: P. Taylor Goetz <pt...@gmail.com>
Date: 2014-03-26T16:32:47Z
STORM-263: update to carbonite 1.4.0/kryo 2.21
----
---
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] incubator-storm pull request: STORM-263: update to carbonite 1.4.0...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-storm/pull/56
---
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] incubator-storm pull request: STORM-263: update to carbonite 1.4.0...
Posted by sritchie <gi...@git.apache.org>.
Github user sritchie commented on the pull request:
https://github.com/apache/incubator-storm/pull/56#issuecomment-39215279
+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] incubator-storm pull request: STORM-263: update to carbonite 1.4.0...
Posted by revans2 <gi...@git.apache.org>.
Github user revans2 commented on the pull request:
https://github.com/apache/incubator-storm/pull/56#issuecomment-39135087
+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] incubator-storm pull request: STORM-263: update to carbonite 1.4.0...
Posted by d2r <gi...@git.apache.org>.
Github user d2r commented on the pull request:
https://github.com/apache/incubator-storm/pull/56#issuecomment-39215928
+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] incubator-storm pull request: STORM-263: update to carbonite 1.4.0...
Posted by sorenmacbeth <gi...@git.apache.org>.
Github user sorenmacbeth commented on the pull request:
https://github.com/apache/incubator-storm/pull/56#issuecomment-39217970
+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] incubator-storm pull request: STORM-263: update to carbonite 1.4.0...
Posted by sritchie <gi...@git.apache.org>.
Github user sritchie commented on the pull request:
https://github.com/apache/incubator-storm/pull/56#issuecomment-39212695
Who has the authority to merge this? Six days without comment... @nathanmarz, is this still your project?
---
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] incubator-storm pull request: STORM-263: update to carbonite 1.4.0...
Posted by nathanmarz <gi...@git.apache.org>.
Github user nathanmarz commented on the pull request:
https://github.com/apache/incubator-storm/pull/56#issuecomment-39248192
As you can see from the location of the project, it's now an Apache project. Like all Apache projects, any committer has authority to merge. The rule we're using on Apache Storm is patches can be merged with at least 2 +1's from committers (who didn't post the patch) and no -1's from committers. Of course, sufficient time needs to be allowed for committers to give their +1 or -1's.
---
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.
---