[GitHub] storm pull request: [STORM-639] storm-maven-plugin not found

2015-01-31 Thread Lewuathe
Github user Lewuathe commented on the pull request: https://github.com/apache/storm/pull/403#issuecomment-72318223 @d2r Thank you for reporting. I'll fix to succeed compile with `mvn compile`. --- If your project is set up for it, you can reply to this email and have your reply

[jira] [Commented] (STORM-639) storm-maven-plugin not found

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

[jira] [Commented] (STORM-609) add storm-redis to storm external

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

[GitHub] storm pull request: [STORM-609] Add storm-redis to storm external

2015-01-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/365#issuecomment-72309792 I wish to hear additional opinions, and finally see it pulled in master. :) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] storm pull request: Allow parsing of String into int for command l...

2015-01-31 Thread jreichhold
Github user jreichhold commented on the pull request: https://github.com/apache/storm/pull/119#issuecomment-72332687 This is already fixed in head. Closing --- 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

[GitHub] storm pull request: Storm 631: refactoring kafka connector code.

2015-01-31 Thread wurstmeister
Github user wurstmeister commented on the pull request: https://github.com/apache/storm/pull/387#issuecomment-72336141 Hi thanks for starting this refactoring, i've had this on my list of things to do but unfortunately have not had the time to do it. A few

[GitHub] storm pull request: Storm 631: refactoring kafka connector code.

2015-01-31 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on the pull request: https://github.com/apache/storm/pull/387#issuecomment-72349915 Thanks for the suggestions, I have tried to address all the concerns below. * I can create multiple jiras and separate the refactoring in two parts as you