[GitHub] storm issue #1679: STORM-2087: storm-kafka-client - tuples not always being ...

2016-11-18 Thread harshach
Github user harshach commented on the issue: https://github.com/apache/storm/pull/1679 Thanks for the patch @jfenc91 and for being patient with reviews. Merged into master. I would like to merge this into 1.x-branch as well. There are few issues on SingleTopicKafkaSpoutTest.java

[GitHub] storm pull request #1679: STORM-2087: storm-kafka-client - tuples not always...

2016-11-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1679 --- 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

RE: NullPointerException on startup

2016-11-18 Thread Cuneo, Nicholas
I’m having trouble finding that version, is it located in a separate repository? Thanks, Nick From: P. Taylor Goetz [mailto:ptgo...@gmail.com] Sent: Friday, November 18, 2016 12:40 PM To: dev@storm.apache.org; Cuneo, Nicholas Subject: Re: NullPointerException on startup (Moving thread to dev@

[GitHub] storm pull request #1786: STORM-1281: LocalCluster, testing4j and testing.cl...

2016-11-18 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1786#discussion_r88749944 --- Diff: storm-core/test/clj/org/apache/storm/clojure_test.clj --- @@ -1,159 +0,0 @@ -;; Licensed to the Apache Software Foundation (ASF) under one

Re: NullPointerException on startup

2016-11-18 Thread P. Taylor Goetz
(Moving thread to dev@ to discuss versions not yet released) Nick, I suspect you might be hitting a bug in 1.0.2 that might be fixed in one of the next releases. Can you try setting the version of storm-kafka-client to 1.1.0-SNAPSHOT (leave other storm components at 1.0.2)? If that fixes the

[GitHub] storm pull request #1786: STORM-1281: LocalCluster, testing4j and testing.cl...

2016-11-18 Thread revans2
GitHub user revans2 opened a pull request: https://github.com/apache/storm/pull/1786 STORM-1281: LocalCluster, testing4j and testing.clj to java This is based off of #1744, but I wanted to make sure that my changes were publicly available. Now Testing.java replaces

[GitHub] storm issue #1739: STORM-1443 [Storm SQL] Support customizing parallelism in...

2016-11-18 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1739 @HeartSaVioR >There might be some ways to pass partition count to upstream, and easy way to do might be adding method to DataSourcesProvider. What I'm considering is that we're now only

[GitHub] storm issue #1742: STORM-2170 [Storm SQL] Add built-in socket datasource to ...

2016-11-18 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1742 OK. nice :smile: --- 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

[GitHub] storm issue #1742: STORM-2170 [Storm SQL] Add built-in socket datasource to ...

2016-11-18 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1742 @vesense Yes I plan to update #1777 after merging this. --- 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

[GitHub] storm issue #1742: STORM-2170 [Storm SQL] Add built-in socket datasource to ...

2016-11-18 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1742 @HeartSaVioR It would be better to add some `how to use` document. --- 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