[GitHub] storm pull request #1581: STORM-1993 Update storm-sql README to have actual ...

[HeartSaVioR <gi...@git.apache.org>]

Github user HeartSaVioR commented on a diff in the pull request:

    https://github.com/apache/storm/pull/1581#discussion_r73668059
  
    --- Diff: external/sql/README.md ---
    @@ -70,17 +70,31 @@ To run this example, users need to include the data sources (`storm-sql-kafka` i
     class path. One approach is to put the required jars into the `extlib` directory:
     
     ```
    -$ cp curator-client-2.5.0.jar curator-framework-2.5.0.jar zookeeper-3.4.6.jar
    - extlib/
    -$ cp scala-library-2.10.4.jar kafka-clients-0.8.2.1.jar kafka_2.10-0.8.2.1.jar metrics-core-2.2.0.jar extlib/
    -$ cp json-simple-1.1.1.jar extlib/
    -$ cp storm-kafka-*.jar storm-sql-kafka-*.jar storm-sql-runtime-*.jar extlib/
    +calcite-avatica-1.4.0-incubating.jar
    --- End diff --
    
    @manuzhang 
    Please refer https://github.com/apache/storm/pull/1608#issuecomment-237807425
    Once we merge STORM-2016 we can get rid of needs for copying dependencies to extlib. When STORM-2023 is merged, running `storm sql` will be much simpler.


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