[jira] [Updated] (STORM-2075) Storm SQL Phase III

2016-08-31 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-2075?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim updated STORM-2075: Description: This epic tracks the effort of the phase III development of StormSQL. For now Storm

[jira] [Commented] (STORM-1444) Support EXPLAIN statement in StormSQL

2016-08-31 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15454308#comment-15454308 ] Jungtaek Lim commented on STORM-1444: - It might need to have different approach. Storm SQL is

[GitHub] storm issue #1665: STORM-2074: fix storm-kafka-monitor NPE bug

2016-08-31 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1665 Thanks @priyank5485 @abellina I think I have addressed your comments. --- 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 issue #1666: STORM-1459: allow not specifying producer properties in r...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1666 Great. Thanks for the quick update. +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

[GitHub] storm issue #1666: STORM-1459: allow not specifying producer properties in r...

2016-08-31 Thread manuzhang
Github user manuzhang commented on the issue: https://github.com/apache/storm/pull/1666 @HeartSaVioR Updated. I use "Kafka sink table" instead of "consumer" to avoid confusing with kafka consumer. And I don't think `TBLPROPERTIES` is an option for "table as a producer" so I leave

[GitHub] storm issue #1648: STORM-2054 DependencyResolver should be aware of relative...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1648 @revans2 This is also a fix for STORM-2059. please review this. --- 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 issue #1660: STORM-2070 Fix sigar native binary download link

2016-08-31 Thread hackerwin7
Github user hackerwin7 commented on the issue: https://github.com/apache/storm/pull/1660 the fix is worked to build --- 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] [Updated] (STORM-2074) NPE bug in storm-kafka-monitor

2016-08-31 Thread Xin Wang (JIRA)
[ https://issues.apache.org/jira/browse/STORM-2074?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xin Wang updated STORM-2074: Description: storm-kafka-monitor will throw an NPE when __--zk-node__ value does not exist. this is not

[GitHub] storm pull request #1665: STORM-2074: fix storm-kafka-monitor NPE bug

2016-08-31 Thread vesense
Github user vesense commented on a diff in the pull request: https://github.com/apache/storm/pull/1665#discussion_r77109600 --- Diff: external/storm-kafka-monitor/src/main/java/org/apache/storm/kafka/monitor/KafkaOffsetLagUtil.java --- @@ -373,16 +377,20 @@ private static Options

[jira] [Created] (STORM-2075) Storm SQL Phase III

2016-08-31 Thread Jungtaek Lim (JIRA)
Jungtaek Lim created STORM-2075: --- Summary: Storm SQL Phase III Key: STORM-2075 URL: https://issues.apache.org/jira/browse/STORM-2075 Project: Apache Storm Issue Type: Epic

[GitHub] storm pull request #1665: STORM-2074: fix storm-kafka-monitor NPE bug

2016-08-31 Thread vesense
Github user vesense commented on a diff in the pull request: https://github.com/apache/storm/pull/1665#discussion_r77107687 --- Diff: external/storm-kafka-monitor/src/main/java/org/apache/storm/kafka/monitor/KafkaOffsetLagUtil.java --- @@ -373,16 +377,20 @@ private static Options

[GitHub] storm issue #1666: STORM-1459: allow not specifying producer properties in r...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1666 +1 except minor documentation issue. --- 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

[GitHub] storm pull request #1666: STORM-1459: allow not specifying producer properti...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1666#discussion_r77107492 --- Diff: docs/storm-sql.md --- @@ -104,21 +104,22 @@ Let's say there is a Kafka stream that represents the transactions of orders. Ea The user can

[GitHub] storm pull request #1665: STORM-2074: fix storm-kafka-monitor NPE bug

2016-08-31 Thread abellina
Github user abellina commented on a diff in the pull request: https://github.com/apache/storm/pull/1665#discussion_r77107294 --- Diff: external/storm-kafka-monitor/src/main/java/org/apache/storm/kafka/monitor/KafkaOffsetLagUtil.java --- @@ -373,16 +377,20 @@ private static

[GitHub] storm pull request #1665: STORM-2074: fix storm-kafka-monitor NPE bug

2016-08-31 Thread vesense
Github user vesense commented on a diff in the pull request: https://github.com/apache/storm/pull/1665#discussion_r77106883 --- Diff: external/storm-kafka-monitor/src/main/java/org/apache/storm/kafka/monitor/KafkaOffsetLagUtil.java --- @@ -89,6 +89,10 @@ public static void main

[jira] [Commented] (STORM-1949) Backpressure can cause spout to stop emitting and stall topology

2016-08-31 Thread Zhuo Liu (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15453966#comment-15453966 ] Zhuo Liu commented on STORM-1949: - Hi [~roshan_naik], [~abellina] has

[GitHub] storm pull request #1666: STORM-1459: allow not specifying producer properti...

2016-08-31 Thread manuzhang
GitHub user manuzhang opened a pull request: https://github.com/apache/storm/pull/1666 STORM-1459: allow not specifying producer properties in read-only Kaf… …ka table in StormSQL You can merge this pull request into a Git repository by running: $ git pull

[GitHub] storm issue #1565: STORM-1970: external project examples refator

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1565 I think we have to make sure that users can run example topologies with local and also remote. (Yes these modules are just examples but 'runnable' examples.) So it might be better to

[GitHub] storm pull request #1565: STORM-1970: external project examples refator

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1565#discussion_r77088437 --- Diff: examples/storm-mqtt-examples/pom.xml --- @@ -24,18 +24,24 @@ storm-mqtt-examples - -org.apache.storm -

[jira] [Resolved] (STORM-2056) Bugs in logviewer

2016-08-31 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-2056?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-2056. - Resolution: Fixed Fix Version/s: 1.0.3 1.1.0 2.0.0

[GitHub] storm pull request #1659: [STORM-2056] - Bugs in logviewer

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1659 --- 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 #1658: [STORM-1646] Fix ExponentialBackoffMsgRetryManager...

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1658 --- 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] [Resolved] (STORM-1646) Intermittent test failures in storm-kafka unit tests

2016-08-31 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1646?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-1646. - Resolution: Fixed Fix Version/s: 1.0.3 1.1.0 2.0.0

[GitHub] storm issue #1662: [STORM-1344] Remove sql command from storm-jdbc build.

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1662 @ppoulosk Could you rebase this into master and squash commits into one? I guess discarding first commit would be good. Also please add STORM-1344 as prefix to commit message since we

[GitHub] storm issue #1662: [STORM-1344] Remove sql command from storm-jdbc build.

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1662 OK. Checked changeset doesn't affect the build result. Nice work. +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

[GitHub] storm issue #1661: [STORM-2071] Add in retry after rebalance in unit test

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1661 From JIRA comment @revans2 said he fixed this issue from Supervisor V2 patch. @revans2 Since we saw builds failing (high chance from my dev.) how about addressing build issue before

[CANCELED] [VOTE] Release Apache Storm 0.10.2 (RC1)

2016-08-31 Thread P. Taylor Goetz
Canceling this vote so the fix for STORM-2042 can be included in the next release. -Taylor > On Aug 26, 2016, at 12:00 PM, Harsha Chintalapani wrote: > > We need to get this patch in for 0.10 release > https://github.com/apache/storm/pull/1645 > > Thanks, > Harsha > > On

Re: [VOTE] Release Apache Storm 0.9.7 (RC1)

2016-08-31 Thread P. Taylor Goetz
This vote is now closed and passes with 3 binding +1 votes, and no +0 or -1 votes. Vote tally: +1: P. Taylor Goetz Jungtaek Lim Bobby Evans -Taylor > On Aug 15, 2016, at 4:10 PM, P. Taylor Goetz wrote: > > This is a call to vote on releasing Apache Storm 0.9.7 (rc1). >

[GitHub] storm pull request #1665: STORM-2074: fix storm-kafka-monitor NPE bug

2016-08-31 Thread priyank5485
Github user priyank5485 commented on a diff in the pull request: https://github.com/apache/storm/pull/1665#discussion_r77034491 --- Diff: external/storm-kafka-monitor/src/main/java/org/apache/storm/kafka/monitor/KafkaOffsetLagUtil.java --- @@ -373,16 +377,20 @@ private static

[GitHub] storm pull request #1665: STORM-2074: fix storm-kafka-monitor NPE bug

2016-08-31 Thread priyank5485
Github user priyank5485 commented on a diff in the pull request: https://github.com/apache/storm/pull/1665#discussion_r77033989 --- Diff: external/storm-kafka-monitor/src/main/java/org/apache/storm/kafka/monitor/KafkaOffsetLagUtil.java --- @@ -89,6 +89,10 @@ public static void

[GitHub] storm issue #1661: [STORM-2071] Add in retry after rebalance in unit test

2016-08-31 Thread abellina
Github user abellina commented on the issue: https://github.com/apache/storm/pull/1661 +1 thanks @ppoulosk --- 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

[GitHub] storm issue #1565: STORM-1970: external project examples refator

2016-08-31 Thread abhishekagarwal87
Github user abhishekagarwal87 commented on the issue: https://github.com/apache/storm/pull/1565 +1 Post the conflicts merge. @HeartSaVioR - can you also take a look? --- 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 issue #1665: STORM-2074: fix storm-kafka-monitor NPE bug

2016-08-31 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1665 this should be merged into 1.x-branch. --- 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

[GitHub] storm issue #1642: STORM-2018: Supervisor V2.

2016-08-31 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1642 @HeartSaVioR take your time. I want to be sure that I have plenty of eyeballs looking at this. We are doing this mostly because we started to run into a lot of race conditions in the supervisor.

[GitHub] storm pull request #1665: STORM-2074: fix storm-kafka-monitor NPE bug

2016-08-31 Thread vesense
GitHub user vesense opened a pull request: https://github.com/apache/storm/pull/1665 STORM-2074: fix storm-kafka-monitor NPE bug https://issues.apache.org/jira/browse/STORM-2074 You can merge this pull request into a Git repository by running: $ git pull

[GitHub] storm issue #1662: [STORM-1344] Add check for table

2016-08-31 Thread ppoulosk
Github user ppoulosk commented on the issue: https://github.com/apache/storm/pull/1662 Fixed. Re-ran tests. Thanks. --- 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 issue #1642: STORM-2018: Supervisor V2.

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1642 I'm still reviewing this. Nice work to refactor to introduce greater readability. Will leave comment here once I'm done with first pass. --- If your project is set up for it, you can reply to

[GitHub] storm issue #1662: [STORM-1344] Add check for table

2016-08-31 Thread ppoulosk
Github user ppoulosk commented on the issue: https://github.com/apache/storm/pull/1662 @HeartSaVioR, @abellina, will fix as you suggest. --- 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

[GitHub] storm issue #1661: [STORM-2071] Add in retry after rebalance in unit test

2016-08-31 Thread ppoulosk
Github user ppoulosk commented on the issue: https://github.com/apache/storm/pull/1661 Coded fix, retested, and pushed. Thanks!! --- 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

[GitHub] storm pull request #1642: STORM-2018: Supervisor V2.

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1642#discussion_r77000695 --- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/BasicContainer.java --- @@ -0,0 +1,628 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request #1661: [STORM-2071] Add in retry after rebalance in unit ...

2016-08-31 Thread ppoulosk
Github user ppoulosk commented on a diff in the pull request: https://github.com/apache/storm/pull/1661#discussion_r76999135 --- Diff: storm-core/test/clj/org/apache/storm/nimbus_test.clj --- @@ -1218,8 +1218,16 @@ (submit-local-topology nimbus "t1" {} topology)

[GitHub] storm pull request #1642: STORM-2018: Supervisor V2.

2016-08-31 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1642#discussion_r76997296 --- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/AdvancedFSOps.java --- @@ -0,0 +1,319 @@ +/** + * Licensed to the Apache Software

[jira] [Created] (STORM-2074) NPE bug in storm-kafka-monitor

2016-08-31 Thread Xin Wang (JIRA)
Xin Wang created STORM-2074: --- Summary: NPE bug in storm-kafka-monitor Key: STORM-2074 URL: https://issues.apache.org/jira/browse/STORM-2074 Project: Apache Storm Issue Type: Bug

[GitHub] storm pull request #1642: STORM-2018: Supervisor V2.

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1642#discussion_r76991771 --- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/AdvancedFSOps.java --- @@ -0,0 +1,319 @@ +/** + * Licensed to the Apache Software

[GitHub] storm issue #1660: STORM-2070 Fix sigar native binary download link

2016-08-31 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1660 +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 wishes so, or if the feature

[GitHub] storm pull request #1638: minor: fix tooltip style

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1638 --- 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-2071) nimbus-test test-leadership failing with Exception

2016-08-31 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-2071?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15452246#comment-15452246 ] Robert Joseph Evans commented on STORM-2071: I replaced the sleep with {code} (defn

[GitHub] storm issue #1638: minor: fix tooltip style

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1638 +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 wishes so, or if the

[GitHub] storm issue #1565: STORM-1970: external project examples refator

2016-08-31 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1565 @abhishekagarwal87 is this should be continue? --- 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

[jira] [Commented] (STORM-2071) nimbus-test test-leadership failing with Exception

2016-08-31 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-2071?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15452239#comment-15452239 ] Robert Joseph Evans commented on STORM-2071: I found and fixed this in my supervisor v2 patch.

[GitHub] storm issue #1638: minor: fix tooltip style

2016-08-31 Thread abellina
Github user abellina commented on the issue: https://github.com/apache/storm/pull/1638 👍 --- 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

[jira] [Assigned] (STORM-2071) nimbus-test test-leadership failing with Exception

2016-08-31 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-2071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Joseph Evans reassigned STORM-2071: -- Assignee: Robert Joseph Evans > nimbus-test test-leadership failing with

[GitHub] storm issue #1603: STORM-2012: Upgrade Kafka to 0.8.2.2

2016-08-31 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1603 @darionyaphet two important fixes for snappy compression. please take a look the above link first, thanks. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] storm issue #1638: minor: fix tooltip style

2016-08-31 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1638 hi @HeartSaVioR Done. --- 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

[jira] [Commented] (STORM-1344) storm-jdbc build error "object name already exists: USER_DETAILS in statement"

2016-08-31 Thread Robert Joseph Evans (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15452218#comment-15452218 ] Robert Joseph Evans commented on STORM-1344: Do you have a patch for that? I can see it being

Assignment of threads to tcp ports

2016-08-31 Thread Walid Aljoby
Hi everyone, Is it easier to specify in which tcp port the tasks (threads) will be assigned?Any API can help to do that? Thank You--Regards

[jira] [Resolved] (STORM-2039) Backpressure refactoring in worker and executor

2016-08-31 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-2039?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-2039. - Resolution: Fixed Fix Version/s: 1.0.3 1.1.0 2.0.0

[GitHub] storm pull request #1627: STORM-2039: backpressure refactoring in worker and...

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1627 --- 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 #1632: Storm 2039 backpressure refactoring in worker and ...

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1632 --- 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 #1627: STORM-2039: backpressure refactoring in worker and execut...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1627 +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 wishes so, or if the

[GitHub] storm issue #1638: minor: fix tooltip style

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1638 @vesense Nice finding. I'll merge once you rebase this to current master. 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 issue #1613: Fix emitting of tuples from bolts with Python 3

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1613 @ThijsMergaert Sorry to comment too late. I'm not sure how much python users use Storm python multi-lang directly, and what's more preferred version, 2 vs 3. If many users still

[GitHub] storm pull request #1614: STORM-1890: looks like too many calls were moved t...

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1614 --- 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 #1615: STORM-1890: looks like too many calls were moved t...

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1615 --- 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 #1612: STORM-1890: looks like too many calls were moved t...

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1612 --- 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 #1615: STORM-1890: looks like too many calls were moved to getSt...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1615 +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 wishes so, or if the

[GitHub] storm issue #1612: STORM-1890: looks like too many calls were moved to getSt...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1612 +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 wishes so, or if the

[GitHub] storm issue #1614: STORM-1890: looks like too many calls were moved to getSt...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1614 +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 wishes so, or if the

[jira] [Updated] (STORM-2042) Nimbus client connections not closed properly causing connection leaks

2016-08-31 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-2042?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim updated STORM-2042: Fix Version/s: 0.10.3 > Nimbus client connections not closed properly causing connection leaks >

[GitHub] storm pull request #1645: [STORM-2042] Nimbus client connections not closed ...

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

Re: Question regarding design of the Trident (operations)

2016-08-31 Thread Jungtaek Lim
FYI: Just worked to have one - http://issues.apache.org/jira/browse/STORM-2072 This adds map and flatMap to have additional parameter: outputFields. If outputFields are specified, it replaces origin fields with output fields. - Jungtaek Lim (HeartSaVioR) 2016년 8월 31일 (수) 오전 10:27, Jungtaek Lim

[jira] [Resolved] (STORM-2064) Add storm name and function, access result and function to log-thrift-access

2016-08-31 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-2064?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-2064. - Resolution: Fixed Fix Version/s: 1.0.3 1.1.0 2.0.0

[GitHub] storm pull request #1651: STORM-2064: add function, storm name and auth resu...

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1651 --- 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 #1650: STORM-2064: add function, storm name and auth resu...

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1650 --- 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 #1480: STORM-1881: Create fat-jar of storm-redis which contains ...

2016-08-31 Thread sakanaou
Github user sakanaou commented on the issue: https://github.com/apache/storm/pull/1480 Closed as requested due to STORM-2016. --- 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

[GitHub] storm pull request #1480: STORM-1881: Create fat-jar of storm-redis which co...

2016-08-31 Thread sakanaou
Github user sakanaou closed the pull request at: https://github.com/apache/storm/pull/1480 --- 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 #1645: [STORM-2042] Nimbus client connections not closed properl...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1645 +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 wishes so, or if the

[GitHub] storm issue #1650: STORM-2064: add function, storm name and auth result to t...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1650 +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 wishes so, or if the

[GitHub] storm issue #1651: STORM-2064: add function, storm name and auth result to t...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1651 +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 wishes so, or if the

[jira] [Resolved] (STORM-2063) Add thread name in worker logs

2016-08-31 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-2063?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-2063. - Resolution: Fixed Fix Version/s: 1.0.3 1.1.0 2.0.0

[GitHub] storm pull request #1653: STORM-2063: turn on thread name in worker.xml

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1653 --- 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 #1652: Storm 2063 thread name in worker log improvements

2016-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1652 --- 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] [Closed] (STORM-1703) In local mode, process is not shutting down clearly

2016-08-31 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-1703?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim closed STORM-1703. --- Resolution: Duplicate > In local mode, process is not shutting down clearly >

[GitHub] storm issue #1659: [STORM-2056] - Bugs in logviewer

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1659 +1 Nice work. --- 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

[GitHub] storm issue #1662: [STORM-1344] Add check for table

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1662 @abellina I got what you are saying. @ppoulosk Let's remove test.sql and relevant setting from pom.xml since JdbcClientTest is already in charge to create table. --- If your project is set

[GitHub] storm issue #1661: [STORM-2071] Add in retry after rebalance in unit test

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1661 +1 once comments from @abellina are addressed. Nice finding. --- 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 #1661: [STORM-2071] Add in retry after rebalance in unit ...

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1661#discussion_r76932681 --- Diff: storm-core/test/clj/org/apache/storm/nimbus_test.clj --- @@ -1218,8 +1218,16 @@ (submit-local-topology nimbus "t1" {}

[GitHub] storm issue #1662: [STORM-1344] Add check for table

2016-08-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1662 We need to ensure that created table from unit tests will be dropped when cleaning up tests. Removing data directory would be also OK. --- If your project is set up for it, you can reply to