[GitHub] storm pull request: STORM-960 HiveBolt should ack tuples only afte...

2015-07-30 Thread dossett
Github user dossett commented on the pull request: https://github.com/apache/storm/pull/653#issuecomment-126317166 Committed changes per feedback from @harshach --- 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

[jira] [Commented] (STORM-951) Storm Hive connector leaking connections

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

[jira] [Created] (STORM-967) Unable to package storm-kafka storm-hdfs connectors in a topology

2015-07-30 Thread Sriharsha Chintalapani (JIRA)
Sriharsha Chintalapani created STORM-967: Summary: Unable to package storm-kafka storm-hdfs connectors in a topology Key: STORM-967 URL: https://issues.apache.org/jira/browse/STORM-967

[jira] [Commented] (STORM-938) storm-hive add a time interval to flush tuples to hive

2015-07-30 Thread Aaron Dossett (JIRA)
[ https://issues.apache.org/jira/browse/STORM-938?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14647603#comment-14647603 ] Aaron Dossett commented on STORM-938: - Thanks [~ptgoetz], PR is attached to this

[GitHub] storm pull request: STORM-960 HiveBolt should ack tuples only afte...

2015-07-30 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/653#issuecomment-126338285 +1 @dossett can you up merge the changes with the branch. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[jira] [Commented] (STORM-960) Hive-Bolt can lose tuples when flushing data

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

[jira] [Commented] (STORM-960) Hive-Bolt can lose tuples when flushing data

2015-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-960?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14647594#comment-14647594 ] ASF GitHub Bot commented on STORM-960: -- Github user dossett commented on the pull

Re: Submitting multiple jars to a topology classpath

2015-07-30 Thread Matthias J. Sax
Storm itself does not provide any support to upload multiple jars. As a workaround you can put required jars into $STORM/lib folder manually (you need to do this on every node in the cluster!) -Matthias On 07/30/2015 08:55 AM, Abhishek Agarwal wrote: Currently, as far as I know one has to

[GitHub] storm pull request: STORM-951. Storm Hive connector leaking connec...

2015-07-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/645 --- 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] [Commented] (STORM-945) DefaultRolloverStrategy element is not a policy,and should not be putted in the Policies element.

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

[jira] [Updated] (STORM-945) DefaultRolloverStrategy element is not a policy,and should not be putted in the Policies element.

2015-07-30 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-945?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim updated STORM-945: --- Affects Version/s: (was: 0.11.0) 0.10.0 DefaultRolloverStrategy element is

[jira] [Commented] (STORM-945) DefaultRolloverStrategy element is not a policy,and should not be putted in the Policies element.

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

[GitHub] storm pull request: STORM-945:DefaultRolloverStrategy element is...

2015-07-30 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/638#issuecomment-126351254 +1. I confirmed that works with 0.11.x-SNAPSHOT. Since it is misconfiguration of log4j2 and it's introduced to 0.10.0-beta, we'd be better to apply it to

[DISCUSSION] packaging storm connenctors

2015-07-30 Thread Sriharsha Chintalapani
Hi All, Currently the way we publish storm connector jars into maven repositories is to just publish storm-kafka, hive, hbase without any of its dependencies included. The expectation here is user will include their version of hdfs and kafka dependencies along with storm-hdfs or

[jira] [Updated] (STORM-967) Unable to package storm-kafka storm-hdfs connectors in a topology

2015-07-30 Thread Sriharsha Chintalapani (JIRA)
[ https://issues.apache.org/jira/browse/STORM-967?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sriharsha Chintalapani updated STORM-967: - Assignee: Priyank Shah Unable to package storm-kafka storm-hdfs connectors in a

[jira] [Resolved] (STORM-945) DefaultRolloverStrategy element is not a policy,and should not be putted in the Policies element.

2015-07-30 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-945?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-945. Resolution: Fixed Fix Version/s: 0.10.0 Thanks [~caofangkun] I merged it to master and

[GitHub] storm pull request: STORM-945:DefaultRolloverStrategy element is...

2015-07-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/638 --- 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] [Commented] (STORM-950) Apache Storm website redesign

2015-07-30 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-950?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14648423#comment-14648423 ] Jungtaek Lim commented on STORM-950: Will you post new PR? Or closing newest PR is by

[GitHub] storm pull request: STORM-966 ConfigValidation.DoubleValidator is ...

2015-07-30 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/658#issuecomment-126513135 You're right about DoubleValidator. DoubleValidator is not properly implemented. Btw, Number.class seems not the proper thing to rely, too.

[jira] [Commented] (STORM-966) ConfigValidation.DoubleValidator doesn't really validate whether the type of the object is a double

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

[jira] [Resolved] (STORM-925) Move DefaultRolloverStrategy in log4j2/worker.xml

2015-07-30 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-925?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-925. Resolution: Duplicate Move DefaultRolloverStrategy in log4j2/worker.xml

[jira] [Commented] (STORM-933) NullPointerException during KafkaSpout deactivation

2015-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-933?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14648657#comment-14648657 ] ASF GitHub Bot commented on STORM-933: -- GitHub user caofangkun opened a pull request:

[GitHub] storm pull request: STORM-933:NullPointerException during KafkaSpo...

2015-07-30 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/660 STORM-933:NullPointerException during KafkaSpout deactivation You can merge this pull request into a Git repository by running: $ git pull https://github.com/caofangkun/apache-storm

[jira] [Assigned] (STORM-933) NullPointerException during KafkaSpout deactivation

2015-07-30 Thread caofangkun (JIRA)
[ https://issues.apache.org/jira/browse/STORM-933?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] caofangkun reassigned STORM-933: Assignee: caofangkun NullPointerException during KafkaSpout deactivation

[jira] [Commented] (STORM-950) Apache Storm website redesign

2015-07-30 Thread Sriharsha Chintalapani (JIRA)
[ https://issues.apache.org/jira/browse/STORM-950?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14648696#comment-14648696 ] Sriharsha Chintalapani commented on STORM-950: -- [~kabhwan] will post a new PR.

[GitHub] storm pull request: [STORM-837] Support for exactly once semantics...

2015-07-30 Thread arunmahadevan
Github user arunmahadevan commented on the pull request: https://github.com/apache/storm/pull/644#issuecomment-126570170 @revans2 @ptgoetz @harshach can we go ahead by automatically disabling exactly once for time based rotation and for file sizes threshold, support exactly once for

[jira] [Commented] (STORM-966) ConfigValidation.DoubleValidator doesn't really validate whether the type of the object is a double

2015-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14648788#comment-14648788 ] ASF GitHub Bot commented on STORM-966: -- Github user jerrypeng commented on the pull

[GitHub] storm pull request: STORM-966 ConfigValidation.DoubleValidator is ...

2015-07-30 Thread jerrypeng
Github user jerrypeng commented on the pull request: https://github.com/apache/storm/pull/658#issuecomment-126573508 Perhaps for the purposes of the config topology.stats.sample.rate and future number based configs we should have a PositiveNumberValidator. There is already a

[jira] [Commented] (STORM-837) HdfsState ignores commits

2015-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14648768#comment-14648768 ] ASF GitHub Bot commented on STORM-837: -- Github user arunmahadevan commented on the

[GitHub] storm pull request: STORM-966 ConfigValidation.DoubleValidator is ...

2015-07-30 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/658#issuecomment-126202559 +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

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-30 Thread harshach
Github user harshach closed the pull request at: https://github.com/apache/storm/pull/654 --- 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] [Commented] (STORM-950) Apache Storm website redesign

2015-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-950?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14648067#comment-14648067 ] ASF GitHub Bot commented on STORM-950: -- GitHub user harshach opened a pull request:

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-30 Thread harshach
GitHub user harshach opened a pull request: https://github.com/apache/storm/pull/659 STORM-950. Apache Storm website redesign. You can merge this pull request into a Git repository by running: $ git pull https://github.com/harshach/incubator-storm STORM-950-V4 Alternatively

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-30 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/659#issuecomment-126424522 @HeartSaVioR your comments from previous one addressed in this one https://github.com/apache/storm/pull/654#issuecomment-125104496 . Please take a look --- If your

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-30 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/659#issuecomment-126424575 @ptgoetz please take a look as well. --- 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

[jira] [Commented] (STORM-950) Apache Storm website redesign

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

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-30 Thread harshach
Github user harshach closed the pull request at: https://github.com/apache/storm/pull/659 --- 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] [Commented] (STORM-950) Apache Storm website redesign

2015-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-950?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14648102#comment-14648102 ] ASF GitHub Bot commented on STORM-950: -- Github user harshach closed the pull request

[jira] [Commented] (STORM-966) ConfigValidation.DoubleValidator doesn't really validate whether the type of the object is a double

2015-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14647264#comment-14647264 ] ASF GitHub Bot commented on STORM-966: -- Github user caofangkun commented on the pull

Submitting multiple jars to a topology classpath

2015-07-30 Thread Abhishek Agarwal
Currently, as far as I know one has to package all the dependencies into one jar and then submit it along with topology class. StormSubmitter interface also allows only one jar. Is there any particular reason for this limitation? We have a use case where we want to upload more than one jar

[GitHub] storm pull request: STORM-960 HiveBolt should ack tuples only afte...

2015-07-30 Thread dossett
Github user dossett commented on the pull request: https://github.com/apache/storm/pull/653#issuecomment-126379477 PR rebased and squashed --- 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

[jira] [Commented] (STORM-960) Hive-Bolt can lose tuples when flushing data

2015-07-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-960?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14647860#comment-14647860 ] ASF GitHub Bot commented on STORM-960: -- Github user dossett commented on the pull

[jira] [Created] (STORM-968) Display the stream id in the Trident UI for the Spouts

2015-07-30 Thread Nikhil Singh (JIRA)
Nikhil Singh created STORM-968: -- Summary: Display the stream id in the Trident UI for the Spouts Key: STORM-968 URL: https://issues.apache.org/jira/browse/STORM-968 Project: Apache Storm Issue