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

[GitHub] incubator-storm pull request: [STORM-234] Thrift purpose conf to l...

GitHub user revans2 opened a pull request:

    https://github.com/apache/incubator-storm/pull/109

    [STORM-234] Thrift purpose conf to let DRPC support authentication

    

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

    $ git pull https://github.com/revans2/incubator-storm thrift-purpose

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

    https://github.com/apache/incubator-storm/pull/109.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 #109
    
----
commit bddde9022ebb3e7e5cb1575e29d3c9fa2db23472
Author: Robert (Bobby) Evans <ev...@yahoo-inc.com>
Date:   2014-05-08T19:42:51Z

    Add thrift purpose for better configuration handling.

commit d69ad406ade961b4681eaadb8e23df71629e65b7
Author: Robert (Bobby) Evans <ev...@yahoo-inc.com>
Date:   2014-05-08T20:19:09Z

    Refactored purpose into Connection Type.

commit 9fe5e3a90cbf800c599c5bfc62b56eb547645807
Author: Robert (Bobby) Evans <ev...@yahoo-inc.com>
Date:   2014-05-09T16:11:46Z

    Updated thrift puprose and refactored it to make it cleaner.

----


---
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-234] Thrift purpose conf to l...

Posted by revans2 <gi...@git.apache.org>.
Github user revans2 commented on the pull request:

    https://github.com/apache/incubator-storm/pull/109#issuecomment-42718731
  
    After discussing with a few others I think we are going to try and do a feature branch instead of pushing things in piecemeal.


---
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-234] Thrift purpose conf to l...

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

    https://github.com/apache/incubator-storm/pull/109


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