You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by jmartell7 <gi...@git.apache.org> on 2015/10/25 17:04:38 UTC
[GitHub] storm pull request: Improve documentation including incorrect Kryo...
GitHub user jmartell7 opened a pull request:
https://github.com/apache/storm/pull/817
Improve documentation including incorrect Kryo ser. framework docs
Improve documentation including incorrect serialization framework issue described in STORM-566.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jmartell7/storm STORM-566-trident-spout-docs
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/817.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 #817
----
commit ce90df5f7cb8ad35fdea2a39bf1c0f39f0b2eff5
Author: jmartell7 <jo...@gmail.com>
Date: 2015-10-25T15:58:48Z
Improve documentation including incorrect Kryo serialization framework doc
Improve documentation including incorrect serialization framework issue described in STORM-566.
----
---
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: Improve documentation including incorrect Kryo...
Posted by zhuoliu <gi...@git.apache.org>.
Github user zhuoliu commented on the pull request:
https://github.com/apache/storm/pull/817#issuecomment-151154642
Looks good. +1.
And we may add "[STORM-566]" to the beginning of the pull request's title.
---
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: Improve documentation including incorrect Kryo...
Posted by revans2 <gi...@git.apache.org>.
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/817#issuecomment-151195413
+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 pull request: Improve documentation including incorrect Kryo...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/817
---
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: Improve documentation including incorrect Kryo...
Posted by redsanket <gi...@git.apache.org>.
Github user redsanket commented on the pull request:
https://github.com/apache/storm/pull/817#issuecomment-151188865
Looks good to me +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.
---