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

----


---

Reply via email to