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

Posted to dev@storm.apache.org by aqeelvn <gi...@git.apache.org> on 2018/01/09 18:25:21 UTC

[GitHub] storm pull request #2508: Updated how to get the storm-starter code

GitHub user aqeelvn opened a pull request:

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

    Updated how to get the storm-starter code

    A typical use case for somebody trying this is, first setting up a working storm installation and then running the example. If the storm-starter code is cloned from github directly, it may not match the version of storm they just installed, and giving rise to an error like `Error: Could not find or load main class org.apache.storm.starter.ExclamationTopology` when the topology is submitted.
    
    The README doesn't make it clear that the storm-starter code is also available inside the version they had downloaded from http://storm.apache.org/downloads.html
    
    Hopefully this edit will bring more clarity.

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

    $ git pull https://github.com/aqeelvn/storm patch-1

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

    https://github.com/apache/storm/pull/2508.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 #2508
    
----
commit 9c9aee763c51d999fae50e09620180a51357cae7
Author: Aqeel Nazeer <aq...@...>
Date:   2018-01-09T18:24:35Z

    Updated how to get the storm-starter code
    
    A typical use case for somebody trying this is, first setting up a working storm installation and then running the example. If the storm-starter code is cloned from github directly, it may not match the version of storm they just installed, and giving rise to an error like `Error: Could not find or load main class org.apache.storm.starter.ExclamationTopology` when the topology is submitted.
    
    The README doesn't make it clear that the storm-starter code is also available inside the version they had downloaded from http://storm.apache.org/downloads.html
    
    Hopefully this edit will bring more clarity.

----


---

[GitHub] storm issue #2508: Updated how to get the storm-starter code

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

Github user srdo commented on the issue:

    https://github.com/apache/storm/pull/2508
  
    +1


---

[GitHub] storm pull request #2508: Updated how to get the storm-starter code

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

Github user asfgit closed the pull request at:

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


---

[GitHub] storm issue #2508: Updated how to get the storm-starter code

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

Github user srdo commented on the issue:

    https://github.com/apache/storm/pull/2508
  
    Thanks for contributing @aqeelvn. Keep up the good work :)


---