[jira] [Created] (STORM-633) Nimbus - HTTP Error 413 full HEAD if using kerberos authentication

2015-01-20 Thread Kevin Risden (JIRA)
Kevin Risden created STORM-633: -- Summary: Nimbus - HTTP Error 413 full HEAD if using kerberos authentication Key: STORM-633 URL: https://issues.apache.org/jira/browse/STORM-633 Project: Apache Storm

[GitHub] storm pull request: STORM-354: Testing: allow users to pass TEST-T...

2015-01-20 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/122#issuecomment-70688283 https://github.com/apache/storm/blob/master/storm-core/src/clj/backtype/storm/testing.clj#L190-192 Shows that the environment variable STORM_TEST_TIMEOUT_MS, can

[GitHub] storm pull request: STORM-354: Testing: allow users to pass TEST-T...

2015-01-20 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/122#issuecomment-70689189 @revans2 its in DEVELOPER.md https://github.com/apache/storm/blob/master/DEVELOPER.md#build-the-code-and-run-the-tests --- If your project is set up for it, you can

[jira] [Commented] (STORM-632) New grouping for better load balancing

2015-01-20 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14284015#comment-14284015 ] Robert Joseph Evans commented on STORM-632: --- [~azaroth] I would love to see this

[jira] [Commented] (STORM-469) Storm UI Last Error Detail Insufficient for debugging

2015-01-20 Thread Jason Kania (JIRA)
[ https://issues.apache.org/jira/browse/STORM-469?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14284050#comment-14284050 ] Jason Kania commented on STORM-469: --- Such a solution would have helped me significantly.

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

2015-01-20 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/365#discussion_r23232733 --- Diff: external/storm-redis/src/main/java/org/apache/storm/redis/trident/state/RedisClusterMapState.java --- @@ -0,0 +1,295 @@ +/** + * Licensed

[jira] [Commented] (STORM-469) Storm UI Last Error Detail Insufficient for debugging

2015-01-20 Thread Sriharsha Chintalapani (JIRA)
[ https://issues.apache.org/jira/browse/STORM-469?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14283970#comment-14283970 ] Sriharsha Chintalapani commented on STORM-469: -- [~dagit] you are right, looks

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

2015-01-20 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/365#discussion_r23232475 --- Diff: external/storm-redis/pom.xml --- @@ -0,0 +1,108 @@ +?xml version=1.0 encoding=UTF-8? +!-- + Licensed to the Apache Software Foundation

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

2015-01-20 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 resolved STORM-599. --- Resolution: Fixed Fix Version/s: 0.10.0 The fixed version is not in master

[jira] [Commented] (STORM-629) Place Link to Source Code Repository on Webpage

2015-01-20 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14284212#comment-14284212 ] Robert Joseph Evans commented on STORM-629: --- There really are two different

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

2015-01-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/381 --- 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] [Created] (STORM-632) New grouping for better load balancing

2015-01-20 Thread Gianmarco De Francisci Morales (JIRA)
Gianmarco De Francisci Morales created STORM-632: Summary: New grouping for better load balancing Key: STORM-632 URL: https://issues.apache.org/jira/browse/STORM-632 Project: Apache

[jira] [Commented] (STORM-633) Nimbus - HTTP Error 413 full HEAD if using kerberos authentication

2015-01-20 Thread Adam Muise (JIRA)
[ https://issues.apache.org/jira/browse/STORM-633?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14284103#comment-14284103 ] Adam Muise commented on STORM-633: -- The resulting error in the ui.log with kerberos

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

2015-01-20 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/365#discussion_r23264315 --- Diff: external/storm-redis/src/main/java/org/apache/storm/redis/trident/state/RedisClusterMapState.java --- @@ -0,0 +1,295 @@ +/** + *

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

2015-01-20 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on the pull request: https://github.com/apache/storm/pull/365#issuecomment-70752983 @HeartSaVioR and @dashengju Thanks for the patch and I think overall the code looks good. I am little confused about the core bolt. You have a trident

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

2015-01-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/365#issuecomment-70755347 @Parth-Brahmbhatt Thanks for review! First of all, let's start about abstract bolt. https://github.com/apache/storm/pull/365#issuecomment-68332013

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

2015-01-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/365#issuecomment-70752747 @dashengju For your information, Jedis team is discussing about Jedis 3.0, and ShardedJedis could be replaced to JedisCluster for reducing code complexity.

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

2015-01-20 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/365#discussion_r23262904 --- Diff: external/storm-redis/pom.xml --- @@ -0,0 +1,108 @@ +?xml version=1.0 encoding=UTF-8? +!-- + Licensed to the Apache Software

[GitHub] storm pull request: STORM-469

2015-01-20 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/388#issuecomment-70753999 I agree it is definitely a great idea to put the link on the UI, and this is what #150 did when it added `errorWorkerLogLink` to storm. But I am not sure it adds

[jira] [Commented] (STORM-528) examples/storm-starter/multilang/resources/storm.py diverged

2015-01-20 Thread Rekha Joshi (JIRA)
[ https://issues.apache.org/jira/browse/STORM-528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14284559#comment-14284559 ] Rekha Joshi commented on STORM-528: --- https://github.com/apache/storm/pull/389

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

2015-01-20 Thread dashengju
Github user dashengju commented on the pull request: https://github.com/apache/storm/pull/365#issuecomment-70777681 @revans2 , I have added you to the committer. thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] storm pull request: STORM-469

2015-01-20 Thread rekhajoshm
Github user rekhajoshm commented on the pull request: https://github.com/apache/storm/pull/388#issuecomment-70787226 @d2r Thanks, and that is another thing - why do we have log link on port number? That does not make any sense at all.The truncated text is where the user expects a

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

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

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

2015-01-20 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on the pull request: https://github.com/apache/storm/pull/387#issuecomment-70728873 I also tested the changes against kafka 0.8.2 release candidate and was able to verify that bolts/spouts/trident states/emitter are able to read and write from

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

2015-01-20 Thread Parth-Brahmbhatt
GitHub user Parth-Brahmbhatt opened a pull request: https://github.com/apache/storm/pull/387 Storm 631: refactoring kafka connector code. This also covers STORM-590. I could not come up with a better name for partitionCoordinator which is more like PartitionManagerCache but it also