[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15410366#comment-15410366
]
ASF GitHub Bot commented on STORM-1979:
---
Github user satishd commented on the issue:
Github user satishd commented on the issue:
https://github.com/apache/storm/pull/1583
+1 after addressing minor comment in README doc.
---
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
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15410362#comment-15410362
]
ASF GitHub Bot commented on STORM-1979:
---
Github user satishd commented on a diff in the pull
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/1583#discussion_r73778048
--- Diff: external/storm-druid/README.md ---
@@ -0,0 +1,143 @@
+# Storm Druid Bolt and TridentState
+
+This module provides core Storm and Trident
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15410359#comment-15410359
]
ASF GitHub Bot commented on STORM-1979:
---
Github user satishd commented on a diff in the pull
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/1583#discussion_r73777827
--- Diff:
external/storm-druid/src/main/java/org/apache/storm/druid/bolt/DruidBeamBolt.java
---
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache
[
https://issues.apache.org/jira/browse/STORM-1890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15410023#comment-15410023
]
ASF GitHub Bot commented on STORM-1890:
---
GitHub user abellina opened a pull request:
GitHub user abellina opened a pull request:
https://github.com/apache/storm/pull/1615
STORM-1890: looks like too many calls were moved to getStatic. It doeâ¦
â¦snt break things, but it is sloppy, so reverting some to $.get
You can merge this pull request into a Git repository by
[
https://issues.apache.org/jira/browse/STORM-1890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15410013#comment-15410013
]
ASF GitHub Bot commented on STORM-1890:
---
GitHub user abellina opened a pull request:
GitHub user abellina opened a pull request:
https://github.com/apache/storm/pull/1614
STORM-1890: looks like too many calls were moved to getStatic. It doeâ¦
â¦snt break things, but it is sloppy, so reverting some to $.get
You can merge this pull request into a Git repository by
GitHub user ThijsMergaert opened a pull request:
https://github.com/apache/storm/pull/1613
Fix emitting of tuples from bolts with Python 3
In Python 3, the map() function no longer returns a list, but instead an
iterable object, which is not JSON serializable. To fix this, convert
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409913#comment-15409913
]
ASF GitHub Bot commented on STORM-1979:
---
Github user omkreddy commented on a diff in the pull
Github user omkreddy commented on a diff in the pull request:
https://github.com/apache/storm/pull/1583#discussion_r73744692
--- Diff:
external/storm-druid/src/main/java/org/apache/storm/druid/bolt/DruidBeamBolt.java
---
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/1606
---
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
[
https://issues.apache.org/jira/browse/STORM-2020?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Robert Joseph Evans resolved STORM-2020.
Resolution: Fixed
Fix Version/s: 2.0.0
> Stop using sun internal classes
>
[
https://issues.apache.org/jira/browse/STORM-2020?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409907#comment-15409907
]
ASF GitHub Bot commented on STORM-2020:
---
Github user asfgit closed the pull request at:
[
https://issues.apache.org/jira/browse/STORM-2022?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Robert Joseph Evans resolved STORM-2022.
Resolution: Fixed
Fix Version/s: 2.0.0
>
[
https://issues.apache.org/jira/browse/STORM-2021?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Robert Joseph Evans resolved STORM-2021.
Resolution: Fixed
Assignee: Robert Joseph Evans (was: Priyank Shah)
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409893#comment-15409893
]
ASF GitHub Bot commented on STORM-1979:
---
Github user satishd commented on a diff in the pull
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/1583#discussion_r73741669
--- Diff:
external/storm-druid/src/main/java/org/apache/storm/druid/bolt/DruidBeamBolt.java
---
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache
[
https://issues.apache.org/jira/browse/STORM-1890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409891#comment-15409891
]
ASF GitHub Bot commented on STORM-1890:
---
GitHub user abellina opened a pull request:
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/1583#discussion_r73741418
--- Diff:
external/storm-druid/src/main/java/org/apache/storm/druid/bolt/DruidConfig.java
---
@@ -0,0 +1,112 @@
+/*
+ * Licensed to the Apache
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409892#comment-15409892
]
ASF GitHub Bot commented on STORM-1979:
---
Github user satishd commented on a diff in the pull
GitHub user abellina opened a pull request:
https://github.com/apache/storm/pull/1612
STORM-1890: looks like too many calls were moved to getStatic
Reverting some of the changes I made in:
https://github.com/apache/storm/pull/1536
What is on master shouldn't break
Github user omkreddy commented on the issue:
https://github.com/apache/storm/pull/1583
@satishd Thanks for the review. addressed review 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 does not
Github user satishd commented on the issue:
https://github.com/apache/storm/pull/1583
@omkreddy Let me know once the comments are addressed.
---
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
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409744#comment-15409744
]
ASF GitHub Bot commented on STORM-1979:
---
Github user satishd commented on the issue:
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409739#comment-15409739
]
ASF GitHub Bot commented on STORM-1979:
---
Github user satishd commented on a diff in the pull
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/1583#discussion_r73727010
--- Diff:
external/storm-druid/src/main/java/org/apache/storm/druid/bolt/DruidBeamBolt.java
---
@@ -0,0 +1,111 @@
+/*
+ * Licensed to the Apache
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409731#comment-15409731
]
ASF GitHub Bot commented on STORM-1979:
---
Github user satishd commented on a diff in the pull
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/1583#discussion_r73726170
--- Diff:
external/storm-druid/src/main/java/org/apache/storm/druid/bolt/TupleDruidEventMapper.java
---
@@ -0,0 +1,42 @@
+/*
+ * Licensed to the
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409729#comment-15409729
]
ASF GitHub Bot commented on STORM-1979:
---
Github user satishd commented on a diff in the pull
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/1583#discussion_r73725602
--- Diff:
external/storm-druid/src/main/java/org/apache/storm/druid/bolt/TupleDruidEventMapper.java
---
@@ -0,0 +1,42 @@
+/*
+ * Licensed to the
Github user harshach commented on a diff in the pull request:
https://github.com/apache/storm/pull/1583#discussion_r73721068
--- Diff:
external/storm-druid/src/main/java/org/apache/storm/druid/bolt/DruidConfig.java
---
@@ -0,0 +1,114 @@
+/*
+ * Licensed to the Apache
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409672#comment-15409672
]
ASF GitHub Bot commented on STORM-1979:
---
Github user harshach commented on the issue:
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409670#comment-15409670
]
ASF GitHub Bot commented on STORM-1979:
---
Github user harshach commented on a diff in the pull
Github user harshach commented on the issue:
https://github.com/apache/storm/pull/1583
missed @satishd suggestions above. Once those addressed I am +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
Alessandro Bellina created STORM-2024:
-
Summary: Add local store autocomplete to the logger name input
field as a convenience (dynamic log level settings)
Key: STORM-2024
URL:
+1 (binding)
Ran on a 3-node cluster and verified fixes.
-Taylor
> On Jul 26, 2016, at 4:04 PM, P. Taylor Goetz wrote:
>
> This is a call to vote on releasing Apache Storm 1.0.2 (rc4)
>
> Full list of changes in this release:
>
>
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1611
@lujinhong No need to sorry. It's my bad, and we all humans do mistake.
Since @revans2 already addressed this from another pull request (#1606), we
can merge that and close this.
---
If
[
https://issues.apache.org/jira/browse/STORM-2016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409554#comment-15409554
]
ASF GitHub Bot commented on STORM-2016:
---
Github user HeartSaVioR commented on a diff in the pull
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/1608#discussion_r73706186
--- Diff: bin/storm.py ---
@@ -154,6 +157,39 @@ def confvalue(name, extrapaths, daemon=True):
return " ".join(tokens[1:])
return
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1608
@revans2 @satishd
Yes I've excluded that for simplification. Since this is a new feature I'd
love to show that this concept works.
Moreover, in order to do that Nimbus should be modified
[
https://issues.apache.org/jira/browse/STORM-1979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409537#comment-15409537
]
ASF GitHub Bot commented on STORM-1979:
---
Github user harshach commented on the issue:
[
https://issues.apache.org/jira/browse/STORM-2016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409539#comment-15409539
]
ASF GitHub Bot commented on STORM-2016:
---
Github user HeartSaVioR commented on a diff in the pull
[
https://issues.apache.org/jira/browse/STORM-2016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409538#comment-15409538
]
ASF GitHub Bot commented on STORM-2016:
---
Github user HeartSaVioR commented on the issue:
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/1608#discussion_r73704312
--- Diff: bin/storm.py ---
@@ -154,6 +157,39 @@ def confvalue(name, extrapaths, daemon=True):
return " ".join(tokens[1:])
return
Github user harshach commented on the issue:
https://github.com/apache/storm/pull/1583
+1. Nice work @omkreddy .
---
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
[
https://issues.apache.org/jira/browse/STORM-2016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409515#comment-15409515
]
ASF GitHub Bot commented on STORM-2016:
---
Github user satishd commented on the issue:
Github user satishd commented on the issue:
https://github.com/apache/storm/pull/1608
@revans2 This issue was raised in the proposal discussion
[
https://issues.apache.org/jira/browse/STORM-2016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409457#comment-15409457
]
ASF GitHub Bot commented on STORM-2016:
---
Github user revans2 commented on the issue:
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/1608
Conceptually I like the idea, and overall the code looks good. My biggest
concern is around cleanup of the bobs after the topology finishes. I don't see
anywhere in the code that it is doing that,
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1608#discussion_r7369
--- Diff: bin/storm.py ---
@@ -154,6 +157,39 @@ def confvalue(name, extrapaths, daemon=True):
return " ".join(tokens[1:])
return ""
[
https://issues.apache.org/jira/browse/STORM-2016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409446#comment-15409446
]
ASF GitHub Bot commented on STORM-2016:
---
Github user revans2 commented on a diff in the pull
GitHub user lujinhong opened a pull request:
https://github.com/apache/storm/pull/1611
storm-2022: fix test case
Sorry, my bad.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/lujinhong/storm storm-2022
Alternatively you can
Hi devs,
Pull request for this proposal is now available.
https://github.com/apache/storm/pull/1608
It's only against 1.x-branch, and I need to update the doc. I'll take care
of them accordingly.
Please take a look at pull request and comment.
Thanks,
Jungtaek Lim (HeartSaVioR)
2016년 8월 4일
[
https://issues.apache.org/jira/browse/STORM-1993?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409244#comment-15409244
]
ASF GitHub Bot commented on STORM-1993:
---
Github user HeartSaVioR commented on a diff in the pull
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/1581#discussion_r73668059
--- Diff: external/sql/README.md ---
@@ -70,17 +70,31 @@ To run this example, users need to include the data
sources (`storm-sql-kafka` i
class
[
https://issues.apache.org/jira/browse/STORM-2016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409223#comment-15409223
]
ASF GitHub Bot commented on STORM-2016:
---
Github user HeartSaVioR commented on the issue:
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1608
[STORM-2016](http://issues.apache.org/jira/browse/STORM-2016) can also
resolve [STORM-1435](http://issues.apache.org/jira/browse/STORM-1435) via
applying different strategy, specifying
[
https://issues.apache.org/jira/browse/STORM-2023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409216#comment-15409216
]
ASF GitHub Bot commented on STORM-2023:
---
GitHub user HeartSaVioR opened a pull request:
GitHub user HeartSaVioR opened a pull request:
https://github.com/apache/storm/pull/1610
STORM-2023 Add calcite-core to dependency of storm-sql-runtime
* also copy (transitive) dependencies to binary distribute, as
storm-sql-core did
It can be easily applied to 1.x-branch.
[
https://issues.apache.org/jira/browse/STORM-2023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409210#comment-15409210
]
Jungtaek Lim commented on STORM-2023:
-
It also needs to copy dependencies (sure transitive too) to
[
https://issues.apache.org/jira/browse/STORM-2023?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jungtaek Lim updated STORM-2023:
Summary: Add calcite-core to dependency of storm-sql-runtime (was: Add
calcite-core to
[
https://issues.apache.org/jira/browse/STORM-2023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409073#comment-15409073
]
Jungtaek Lim commented on STORM-2023:
-
Compiled topology refers SqlFunctions on calcite-core which
[
https://issues.apache.org/jira/browse/STORM-2016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409069#comment-15409069
]
ASF GitHub Bot commented on STORM-2016:
---
Github user HeartSaVioR commented on the issue:
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1608
Builds for PR against 1.x branch is failing and I fixed that from #1609.
I'll rebase once #1609 is merged to 1.x-branch.
---
If your project is set up for it, you can reply to this email and
[
https://issues.apache.org/jira/browse/STORM-2016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15409034#comment-15409034
]
ASF GitHub Bot commented on STORM-2016:
---
Github user HeartSaVioR commented on the issue:
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1608
Fixed build failure on supervisor_test.
---
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
Jungtaek Lim created STORM-2023:
---
Summary: Add calcite-core to storm-sql-runtime
Key: STORM-2023
URL: https://issues.apache.org/jira/browse/STORM-2023
Project: Apache Storm
Issue Type: Bug
70 matches
Mail list logo