You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by rauluka <gi...@git.apache.org> on 2018/03/24 19:38:08 UTC

[GitHub] storm pull request #2604: STORM-3006-updated-DRPC-docs

GitHub user rauluka opened a pull request:

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

    STORM-3006-updated-DRPC-docs

    Extended DRPCClient initialization with DRPCClient config (which is now mandatory parameter) and added storm.thrift.transport: "org.apache.storm.security.auth.plain.PlainSaslTransportPlugin" in storm.yaml example so it matches DRPCClient config.

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

    $ git pull https://github.com/rauluka/storm 1.x-branch

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

    https://github.com/apache/storm/pull/2604.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 #2604
    
----
commit 65720ac71ad85966cd59306c47c5ac5ab988d34f
Author: Lukasz Gebel <lu...@...>
Date:   2018-03-24T19:30:23Z

    Updated Distributed-RPC.md with new way of DRPCClient initialization and added storm.thrift.transport property settings example.

----


---

[GitHub] storm pull request #2604: STORM-3006-updated-DRPC-docs

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

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


---

[GitHub] storm issue #2604: STORM-3006-updated-DRPC-docs

Posted by revans2 <gi...@git.apache.org>.

Github user revans2 commented on the issue:

    https://github.com/apache/storm/pull/2604
  
    The docs are not identical for master, so I will manually make some updates to master, which is allowed without code review.


---