[GitHub] storm issue #1754: [STORM-2177][SQL] Support TSV as input / output format

2016-10-31 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1754 This PR is on top of https://github.com/apache/storm/pull/1751 --- 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

[GitHub] storm pull request #1754: [STORM-2177][SQL] Support TSV as input / output fo...

2016-10-31 Thread vesense
GitHub user vesense opened a pull request: https://github.com/apache/storm/pull/1754 [STORM-2177][SQL] Support TSV as input / output format You can merge this pull request into a Git repository by running: $ git pull https://github.com/vesense/storm STORM-2177 Alternatively

[GitHub] storm pull request #1753: STORM-2180: KafkaSpout ack() should handle re-bala...

2016-10-31 Thread knusbaum
GitHub user knusbaum opened a pull request: https://github.com/apache/storm/pull/1753 STORM-2180: KafkaSpout ack() should handle re-balance - 1.x branch You can merge this pull request into a Git repository by running: $ git pull https://github.com/knusbaum/incubator-storm

[GitHub] storm pull request #1752: STORM-2180: KafkaSpout ack() should handle re-bala...

2016-10-31 Thread knusbaum
GitHub user knusbaum opened a pull request: https://github.com/apache/storm/pull/1752 STORM-2180: KafkaSpout ack() should handle re-balance You can merge this pull request into a Git repository by running: $ git pull https://github.com/knusbaum/incubator-storm STORM-2180

Re: storm on windows

2016-10-31 Thread Bobby Evans
My understanding was that if the code is a core command then it calls the function https://github.com/apache/storm/blob/1.0.x-branch/bin/storm.cmd#L63-L71 and then goes on with CLASS_PATH etc set already and then here. https://github.com/apache/storm/blob/1.0.x-branch/bin/storm.cmd#L119-L124

Re: storm on windows

2016-10-31 Thread moodysma sma
Also, if we use storm and therefore storm.py, then there is an error on line 92 where the python script checks for java instead of java.exe on Windows. Filed issue 2178 (version problem) and 2179 (java home detection). JAVA_CMD = 'java' if not JAVA_HOME else os.path.join(JAVA_HOME, 'bin',

Re: storm on windows

2016-10-31 Thread moodysma sma
It looks like STORM_DEBUG isn't applicable to "classpath help version" on line 54 so it doesn't do anything. STORM_DEBUG is used on line 119 but a call to jump to :version is made on line 59. set shellcommands=classpath help version for %%i in ( %shellcommands% ) do ( if %storm-command%

Re: storm on windows

2016-10-31 Thread Bobby Evans
Hmm if nothing is being printed out it really feels like it is a problem in the storm.cmd file itself.  Could you set STORM_DEBUG?  It should print out what command is being run. The only other thing I can think of is that some of the config setup code ahead of this is failing some how. -

[GitHub] storm pull request #1749: STORM-2175: Support at least once shutdown in work...

2016-10-31 Thread revans2
GitHub user revans2 reopened a pull request: https://github.com/apache/storm/pull/1749 STORM-2175: Support at least once shutdown in workers Will put up a pull request for 1.x shortly. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] storm pull request #1750: STORM-2175: support at least once shutdown in work...

2016-10-31 Thread revans2
GitHub user revans2 reopened a pull request: https://github.com/apache/storm/pull/1750 STORM-2175: support at least once shutdown in worker You can merge this pull request into a Git repository by running: $ git pull https://github.com/revans2/incubator-storm STORM-2175-1.x

Re: storm on windows

2016-10-31 Thread moodysma sma
I checked it on two different Windows environments: Windows 10, Windows 7. The problem arises when downstream libs like streamparse check the version, they fail because they get back an empty/null string. I'm using storm.cmd. On Mon, Oct 31, 2016 at 11:59 AM, Bobby Evans

Re: storm on windows

2016-10-31 Thread Bobby Evans
That is really odd.  I just downloaded it and got the following on my mac.  It looks like it is a windows bug. $ ./bin/storm version ... Storm 1.0.2 URL https://git-wip-us.apache.org/repos/asf/storm.git -r 54f319fd56c33437364c550a800ac9e6fe058b95 Branch (no branch) Compiled by tgoetz on

Re: storm on windows

2016-10-31 Thread moodysma sma
Hi, I am using the latest prebuilt release: Storm 1.0.2 zip file. Zookeeper 3.4.9. Java 1.8.0_111. http://www.apache.org/dyn/closer.lua/storm/apache-storm-1.0.2/apache-storm-1.0.2.zip On Mon, Oct 31, 2016 at 10:53 AM, Bobby Evans wrote: > What version of storm

[GitHub] storm issue #1750: STORM-2175: support at least once shutdown in worker

2016-10-31 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1750 Will reopen once I have a better fix that gives more time and throws on a timeout. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] storm pull request #1750: STORM-2175: support at least once shutdown in work...

2016-10-31 Thread revans2
Github user revans2 closed the pull request at: https://github.com/apache/storm/pull/1750 --- 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

[GitHub] storm pull request #1749: STORM-2175: Support at least once shutdown in work...

2016-10-31 Thread revans2
Github user revans2 closed the pull request at: https://github.com/apache/storm/pull/1749 --- 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

[GitHub] storm issue #1749: STORM-2175: Support at least once shutdown in workers

2016-10-31 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1749 Will reopen once I have a better fix that gives more time and throws on a timeout. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

Re: storm on windows

2016-10-31 Thread Bobby Evans
What version of storm are you using?  How was it packaged?  In newer versions of storm the version is packaged in a properties file inside the storm-core jar. - Bobby On Sunday, October 30, 2016, 10:35:15 AM CDT, moodysma sma wrote:When I type: "storm version" I get nothing