[jira] [Commented] (STORM-393) Add topic to KafkaSpout output

2015-01-14 Thread Parth Brahmbhatt (JIRA)
[ https://issues.apache.org/jira/browse/STORM-393?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14277604#comment-14277604 ] Parth Brahmbhatt commented on STORM-393: [~ptgoetz] No response in 4 months and as

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22962128 --- Diff: external/storm-jdbc/README.md --- @@ -0,0 +1,208 @@ +#Storm JDBC +Storm/Trident integration for JDBC. This package includes the core

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread harshach
Github user harshach commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22962179 --- Diff: external/storm-jdbc/src/test/java/org/apache/storm/jdbc/topology/AbstractUserTopology.java --- @@ -0,0 +1,102 @@ +/** + * Licensed to the

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread itaifrenkel
Github user itaifrenkel commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22972354 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/bolt/AbstractJdbcBolt.java --- @@ -0,0 +1,53 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/374#issuecomment-69978982 overall I am +1 on merging this. There are minor nit-picks in the above comments. I would like to see the unit tests for the bolt and state. Probably we can use

[jira] [Closed] (STORM-560) ZkHosts in README should use 2181 as port

2015-01-14 Thread Parth Brahmbhatt (JIRA)
[ https://issues.apache.org/jira/browse/STORM-560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Parth Brahmbhatt closed STORM-560. -- Resolution: Duplicate ZkHosts in README should use 2181 as port

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22962079 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/bolt/JdbcBolt.java --- @@ -0,0 +1,91 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22962071 --- Diff: external/storm-jdbc/README.md --- @@ -0,0 +1,208 @@ +#Storm JDBC +Storm/Trident integration for JDBC. This package includes the core

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22962868 --- Diff: external/storm-jdbc/src/test/java/org/apache/storm/jdbc/topology/AbstractUserTopology.java --- @@ -0,0 +1,102 @@ +/** + * Licensed

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22974954 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/bolt/JdbcBolt.java --- @@ -0,0 +1,91 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22973996 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/common/JDBCClient.java --- @@ -0,0 +1,209 @@ +/** + * Licensed to the

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22978604 --- Diff: external/storm-jdbc/README.md --- @@ -0,0 +1,208 @@ +#Storm JDBC +Storm/Trident integration for JDBC. This package includes the core

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22978468 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/common/JDBCClient.java --- @@ -0,0 +1,209 @@ +/** + * Licensed to the

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread itaifrenkel
Github user itaifrenkel commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22973617 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/bolt/JdbcLookupBolt.java --- @@ -0,0 +1,80 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread itaifrenkel
Github user itaifrenkel commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22973557 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/bolt/JdbcLookupBolt.java --- @@ -0,0 +1,80 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread itaifrenkel
Github user itaifrenkel commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22973534 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/bolt/JdbcBolt.java --- @@ -0,0 +1,91 @@ +/** + * Licensed to the Apache Software

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread harshach
Github user harshach commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22974645 --- Diff: external/storm-jdbc/README.md --- @@ -0,0 +1,208 @@ +#Storm JDBC +Storm/Trident integration for JDBC. This package includes the core bolts

[jira] [Updated] (STORM-627) Storm-hbase configuration error

2015-01-14 Thread JIAN ZHOU (JIRA)
[ https://issues.apache.org/jira/browse/STORM-627?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] JIAN ZHOU updated STORM-627: Description: The HBaseMapState and HBaseState classes are not reading the configuration entries properly.

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread itaifrenkel
Github user itaifrenkel commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22973307 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/bolt/JdbcBolt.java --- @@ -0,0 +1,91 @@ +/** + * Licensed to the Apache Software

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread itaifrenkel
Github user itaifrenkel commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22972968 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/common/JDBCClient.java --- @@ -0,0 +1,209 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread itaifrenkel
Github user itaifrenkel commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22975021 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/common/JDBCClient.java --- @@ -0,0 +1,209 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22974973 --- Diff: external/storm-jdbc/src/main/java/org/apache/storm/jdbc/bolt/JdbcLookupBolt.java --- @@ -0,0 +1,80 @@ +/** + * Licensed to the

[jira] [Created] (STORM-628) Storm-HBase add support to WriteBuffer/setAutoFlush

2015-01-14 Thread Weichen Ye (JIRA)
Weichen Ye created STORM-628: Summary: Storm-HBase add support to WriteBuffer/setAutoFlush Key: STORM-628 URL: https://issues.apache.org/jira/browse/STORM-628 Project: Apache Storm Issue Type:

[jira] [Updated] (STORM-628) Storm-HBase add support to WriteBuffer/setAutoFlush

2015-01-14 Thread Weichen Ye (JIRA)
[ https://issues.apache.org/jira/browse/STORM-628?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Weichen Ye updated STORM-628: - Attachment: STORM-628.patch Storm-HBase add support to WriteBuffer/setAutoFlush

[jira] [Commented] (STORM-600) upgrade jacoco plugin to support jdk8

2015-01-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14277060#comment-14277060 ] ASF GitHub Bot commented on STORM-600: -- Github user d2r commented on the pull request:

[GitHub] storm pull request: STORM-600: Update version of jacoco plugin.

2015-01-14 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/357#issuecomment-69932956 +1 I compiled and tested on OSX with jdk 1.8. The build was successful and the tests passed. --- If your project is set up for it, you can reply to this email and have

[jira] [Commented] (STORM-599) UI is slow due to fetching heartbeats from ZK

2015-01-14 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-599?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14277076#comment-14277076 ] Robert Joseph Evans commented on STORM-599: --- [~sriharsha] You are correct I

[GitHub] storm pull request: STORM-600: Update version of jacoco plugin.

2015-01-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/357 --- 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

[jira] [Reopened] (STORM-599) UI is slow due to fetching heartbeats from ZK

2015-01-14 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-599?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans reopened STORM-599: --- As per [~sriharsha]'s comments this has been reverted, and we will explore what is

[jira] [Updated] (STORM-599) UI is slow due to fetching heartbeats from ZK

2015-01-14 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-599?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans updated STORM-599: -- Fix Version/s: (was: 0.10.0) UI is slow due to fetching heartbeats from ZK

[GitHub] storm pull request: STORM-495: KafkaSpout retries with exponential...

2015-01-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/254 --- 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: [STORM-624] Fix typos in SECUTIRY.md

2015-01-14 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on the pull request: https://github.com/apache/storm/pull/379#issuecomment-69945124 +1. --- 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

[jira] [Commented] (STORM-600) upgrade jacoco plugin to support jdk8

2015-01-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14277089#comment-14277089 ] ASF GitHub Bot commented on STORM-600: -- Github user asfgit closed the pull request at:

[jira] [Commented] (STORM-495) Add delayed retries to KafkaSpout

2015-01-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-495?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14277122#comment-14277122 ] ASF GitHub Bot commented on STORM-495: -- Github user asfgit closed the pull request at:

[GitHub] storm pull request: Storm-456: changing encoding to ring/url-encod...

2015-01-14 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/360#issuecomment-69940108 +1 --- 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

[GitHub] storm pull request: STORM-505: Fix debug string construction

2015-01-14 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/359#issuecomment-69940172 +1 --- 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

[jira] [Created] (STORM-625) Netty context never forgets about a Client.

2015-01-14 Thread Robert Joseph Evans (JIRA)
Robert Joseph Evans created STORM-625: - Summary: Netty context never forgets about a Client. Key: STORM-625 URL: https://issues.apache.org/jira/browse/STORM-625 Project: Apache Storm

[jira] [Resolved] (STORM-495) Add delayed retries to KafkaSpout

2015-01-14 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-495?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans resolved STORM-495. --- Resolution: Fixed Fix Version/s: 0.10.0 Assignee: Rick Kilgore Thanks

[jira] [Commented] (STORM-505) Debug Log Message in logviewer causes exception.

2015-01-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-505?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14277127#comment-14277127 ] ASF GitHub Bot commented on STORM-505: -- Github user harshach commented on the pull

[jira] [Commented] (STORM-624) Some typos in SECURITY.md

2015-01-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14277181#comment-14277181 ] ASF GitHub Bot commented on STORM-624: -- Github user Parth-Brahmbhatt commented on the

[GitHub] storm pull request: Storm-616 : Storm-jdbc connector.

2015-01-14 Thread harshach
Github user harshach commented on a diff in the pull request: https://github.com/apache/storm/pull/374#discussion_r22950826 --- Diff: external/storm-jdbc/README.md --- @@ -0,0 +1,208 @@ +#Storm JDBC +Storm/Trident integration for JDBC. This package includes the core bolts

[GitHub] storm pull request: STORM-626: Add script to print out the merge c...

2015-01-14 Thread revans2
GitHub user revans2 opened a pull request: https://github.com/apache/storm/pull/380 STORM-626: Add script to print out the merge command for a given pull request. You can merge this pull request into a Git repository by running: $ git pull

[jira] [Resolved] (STORM-620) Duplicate maven plugin declaration

2015-01-14 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-620?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans resolved STORM-620. --- Resolution: Fixed Fix Version/s: 0.10.0 Thanks [~lewuathe], I merged this into

[jira] [Commented] (STORM-620) Duplicate maven plugin declaration

2015-01-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-620?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14277260#comment-14277260 ] ASF GitHub Bot commented on STORM-620: -- Github user asfgit closed the pull request at:

Re: ScheduledTopology

2015-01-14 Thread Itai Frenkel
Muhammad, We are thinking of using storm for scheduled non-realtime processing (things like you would do with map-reduce once a week or daily). The benefit of using Storm is that the same code could be used in realtime and non-realtime processing (it has some disadvantages ofcourse). For

[jira] [Commented] (STORM-626) merging a pull request can be a pain

2015-01-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-626?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14277238#comment-14277238 ] ASF GitHub Bot commented on STORM-626: -- GitHub user revans2 opened a pull request:

[GitHub] storm pull request: [Storm 599] Use use nimbus's cached heartbeats...

2015-01-14 Thread d2r
GitHub user d2r opened a pull request: https://github.com/apache/storm/pull/381 [Storm 599] Use use nimbus's cached heartbeats rather than fetching again from ZK, take two This is a retry of the reverted pull request #356. In #356, a merge error (we think?) on the branch

[jira] [Commented] (STORM-599) UI is slow due to fetching heartbeats from ZK

2015-01-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-599?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14277273#comment-14277273 ] ASF GitHub Bot commented on STORM-599: -- GitHub user d2r opened a pull request:

[GitHub] storm pull request: [STORM-624] Fix typos in SECUTIRY.md

2015-01-14 Thread Lewuathe
GitHub user Lewuathe opened a pull request: https://github.com/apache/storm/pull/379 [STORM-624] Fix typos in SECUTIRY.md Fix some typos in SECURITY.mf You can merge this pull request into a Git repository by running: $ git pull https://github.com/Lewuathe/storm typo-SECURITY

[jira] [Commented] (STORM-624) Some typos in SECURITY.md

2015-01-14 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14276881#comment-14276881 ] ASF GitHub Bot commented on STORM-624: -- GitHub user Lewuathe opened a pull request:

[jira] [Created] (STORM-624) Some typos in SECURITY.md

2015-01-14 Thread Kai Sasaki (JIRA)
Kai Sasaki created STORM-624: Summary: Some typos in SECURITY.md Key: STORM-624 URL: https://issues.apache.org/jira/browse/STORM-624 Project: Apache Storm Issue Type: Bug Affects Versions:

[jira] [Updated] (STORM-624) Some typos in SECURITY.md

2015-01-14 Thread Kai Sasaki (JIRA)
[ https://issues.apache.org/jira/browse/STORM-624?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kai Sasaki updated STORM-624: - There are some typos in SECURITY.md Some typos in SECURITY.md -