Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76818353
--- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/Slot.java
---
@@ -0,0 +1,769 @@
+/**
+ * Licensed to the Apache Software Foundation
[
https://issues.apache.org/jira/browse/STORM-1985?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449339#comment-15449339
]
Robert Joseph Evans commented on STORM-1985:
Please put the class in org.apache.storm.command.
Hi, I'm the original creator and primary developer of SQE. Sorry for
the radio silence on my part, I was out on vacation the past two
weeks.
I'm glad to see the Storm SQL project chugging along. I started SQE
because I wanted better tools on top of Storm, particularly the
ability to query streams
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76800917
--- Diff:
storm-core/src/jvm/org/apache/storm/daemon/supervisor/BasicContainer.java ---
@@ -0,0 +1,629 @@
+/**
+ * Licensed to the Apache Software
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76819266
--- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/Slot.java
---
@@ -0,0 +1,769 @@
+/**
+ * Licensed to the Apache Software Foundation
GitHub user ppoulosk opened a pull request:
https://github.com/apache/storm/pull/1658
[STORM-1646] Fix ExponentialBackoffMsgRetryManager test
Use simulated time instead of wall clock time in test, as running this in
virutual environments sometime causes failure.
Replaces
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76823287
--- Diff:
storm-core/src/jvm/org/apache/storm/localizer/LocalDownloadedResource.java ---
@@ -0,0 +1,107 @@
+/**
+ * Licensed to the Apache Software
Github user ppoulosk closed the pull request at:
https://github.com/apache/storm/pull/1335
---
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 user ppoulosk commented on the issue:
https://github.com/apache/storm/pull/1335
Replaced by 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
It's traditional. We wait for the last minute to get our talk proposals
in for conferences.
Well, the last minute has arrived. The CFP for ApacheCon Seville closes
on September 9th, which is less than 2 weeks away. It's time to get your
talks in, so that we can make this the best ApacheCon yet.
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76801694
--- Diff:
storm-core/src/jvm/org/apache/storm/daemon/supervisor/BasicContainer.java ---
@@ -0,0 +1,629 @@
+/**
+ * Licensed to the Apache Software
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76812876
--- Diff:
storm-core/src/jvm/org/apache/storm/daemon/supervisor/ReadClusterState.java ---
@@ -0,0 +1,318 @@
+/**
+ * Licensed to the Apache Software
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76802455
--- Diff:
storm-core/src/jvm/org/apache/storm/daemon/supervisor/BasicContainer.java ---
@@ -0,0 +1,629 @@
+/**
+ * Licensed to the Apache Software
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76814090
--- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/Slot.java
---
@@ -0,0 +1,769 @@
+/**
+ * Licensed to the Apache Software Foundation
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76817454
--- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/Slot.java
---
@@ -0,0 +1,769 @@
+/**
+ * Licensed to the Apache Software Foundation
GitHub user ppoulosk reopened a pull request:
https://github.com/apache/storm/pull/1658
[STORM-1646] Fix ExponentialBackoffMsgRetryManager test
Use simulated time instead of wall clock time in test, as running this in
virutual environments sometime causes failure.
Replaces
Github user ppoulosk 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
Github user harshach commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76853245
--- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/Slot.java
---
@@ -0,0 +1,749 @@
+/**
+ * Licensed to the Apache Software Foundation
Hi Morrigan,
Thanks for joining discussion. I thought we need to hear your goal to
donate SQE code, and opinion for how to apply SQE to Storm SQL and working
on further improvements.
Not sure when you took a look at the feature set of Storm SQL, but if you
haven't recently, you may want to do
Hi Manu,
I'm finding for 1:1 tuple transition, not aggregation. In other words, I'm
finding like 'V map(T, Function)' which seems fit for many use case
but Trident doesn't support this yet so I'd like to find out why.
Thanks,
Jungtaek Lim (HeartSaVioR)
2016년 8월 31일 (수) 오전 7:09, Manu Zhang
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76874493
--- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/Slot.java
---
@@ -0,0 +1,769 @@
+/**
+ * Licensed to the Apache Software Foundation
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76873268
--- Diff:
storm-core/src/jvm/org/apache/storm/daemon/supervisor/BasicContainer.java ---
@@ -0,0 +1,629 @@
+/**
+ * Licensed to the Apache Software
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76873417
--- Diff:
storm-core/src/jvm/org/apache/storm/daemon/supervisor/Container.java ---
@@ -0,0 +1,493 @@
+/**
+ * Licensed to the Apache Software
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76875777
--- Diff:
storm-core/src/jvm/org/apache/storm/localizer/LocalDownloadedResource.java ---
@@ -0,0 +1,107 @@
+/**
+ * Licensed to the Apache Software
[
https://issues.apache.org/jira/browse/STORM-2056?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Boyang Jerry Peng updated STORM-2056:
-
Description:
1. Incorrect url for prev,first,last,next buttons when viewing daemon logs
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1658
+1 Nice catch.
---
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,
[
https://issues.apache.org/jira/browse/STORM-2060?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Boyang Jerry Peng closed STORM-2060.
Resolution: Won't Fix
> Consolidate duplicated code in logviewer
>
GitHub user jerrypeng opened a pull request:
https://github.com/apache/storm/pull/1659
[STORM-2056] - Bugs in logviewer
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jerrypeng/storm STORM-2056
Alternatively you can review and
[
https://issues.apache.org/jira/browse/STORM-1282?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sriharsha Chintalapani updated STORM-1282:
--
Assignee: Sivaguru Kannan (was: Abhishek Agarwal)
> port
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76873184
--- Diff:
storm-core/src/jvm/org/apache/storm/daemon/supervisor/BasicContainer.java ---
@@ -0,0 +1,629 @@
+/**
+ * Licensed to the Apache Software
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1642#discussion_r76875102
--- Diff: storm-core/src/jvm/org/apache/storm/daemon/supervisor/Slot.java
---
@@ -0,0 +1,769 @@
+/**
+ * Licensed to the Apache Software Foundation
I think we have 'aggregate' in Trident, where the function output fields
replace the input fields.
Thanks,
Manu Zhang
On Tue, Aug 30, 2016 at 4:17 PM Jungtaek Lim wrote:
> Hi devs,
>
> While implementing some features of Storm SQL on top of Trident, I realized
> that there's
[
https://issues.apache.org/jira/browse/STORM-2071?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15450474#comment-15450474
]
Paul Poulosky commented on STORM-2071:
--
1 second hardcoded timeout isn't enough time to do rebalance.
Github user abellina commented on the issue:
https://github.com/apache/storm/pull/1662
could we remove the file all together? Seems like unit tests are also
creating that table...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1661#discussion_r76916977
--- Diff: storm-core/test/clj/org/apache/storm/nimbus_test.clj ---
@@ -1218,8 +1218,16 @@
(submit-local-topology nimbus "t1" {} topology)
Jungtaek Lim created STORM-2070:
---
Summary: Sigar native binary download link went 404
Key: STORM-2070
URL: https://issues.apache.org/jira/browse/STORM-2070
Project: Apache Storm
Issue Type:
Github user abellina commented on the issue:
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
Jungtaek Lim created STORM-2072:
---
Summary: Add map, flatMap with different outputs (T->V) in Trident
Key: STORM-2072
URL: https://issues.apache.org/jira/browse/STORM-2072
Project: Apache Storm
Paul Poulosky created STORM-2071:
Summary: nimbus-test test-leadership failing with Exception
Key: STORM-2071
URL: https://issues.apache.org/jira/browse/STORM-2071
Project: Apache Storm
FYI: I just found that Stream class has TODO comment above of class:
*// TODO: need to be able to replace existing fields with the function
fields (like Cascading Fields.REPLACE)*
which is written at Aug. 2012, initial importing Trident source code.
According to the TODO comment, it seems to be a
GitHub user HeartSaVioR opened a pull request:
https://github.com/apache/storm/pull/1660
STORM-2070 Fix sigar native binary download link
* previous link goes 404, googlecode seems change archive url
This should be also merged back to 1.x, 1.0.x branches.
You can merge
GitHub user HeartSaVioR opened a pull request:
https://github.com/apache/storm/pull/1663
STORM-2072 Add map, flatMap with different outputs (T->V) in Trident
* Introduce overloading methods of map and flatMap
* Now users can specify new output fields which replace origin
Jungtaek Lim created STORM-2073:
---
Summary: Reduce multi-steps on visitProject into one
Key: STORM-2073
URL: https://issues.apache.org/jira/browse/STORM-2073
Project: Apache Storm
Issue Type:
GitHub user HeartSaVioR opened a pull request:
https://github.com/apache/storm/pull/1664
STORM-2073 Reduce multi-steps on visitProject into one
This change is on top of STORM-2072 (#1663).
In STORM-1434 we revamped the way to build Trident topology for Storm SQL.
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1664
Before
https://docs.google.com/spreadsheets/d/1tTyIxbNafvVw0tQjWjiBQCKDCPbbnfbT5JZ7F5JEBNE/edit#gid=0
After
GitHub user ppoulosk opened a pull request:
https://github.com/apache/storm/pull/1661
[STORM-2071] Add in retry after rebalance in unit test
This retries the command a five times after rebalance, as it was failing on
some systems.
You can merge this pull request into a Git
[
https://issues.apache.org/jira/browse/STORM-1344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15450827#comment-15450827
]
Paul Poulosky commented on STORM-1344:
--
I have been running into this error on multple platforms
Github user abellina commented on a diff in the pull request:
https://github.com/apache/storm/pull/1661#discussion_r76917059
--- Diff: storm-core/test/clj/org/apache/storm/nimbus_test.clj ---
@@ -1218,8 +1218,16 @@
(submit-local-topology nimbus "t1" {} topology)
[
https://issues.apache.org/jira/browse/STORM-2072?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15451030#comment-15451030
]
Jungtaek Lim commented on STORM-2072:
-
And I have use case of MapFunction which needs to be
[
https://issues.apache.org/jira/browse/STORM-2071?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15450474#comment-15450474
]
Paul Poulosky edited comment on STORM-2071 at 8/30/16 11:30 PM:
1 second
Github user ppoulosk commented on the issue:
https://github.com/apache/storm/pull/1658
Ran all unit tests successfully on my box. Looks like travis errors are
unrelated.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
GitHub user ppoulosk opened a pull request:
https://github.com/apache/storm/pull/1662
[STORM-1344] Add check for table
Works around STORM-1344 by ensuring table is not double-created.
You can merge this pull request into a Git repository by running:
$ git pull
[
https://issues.apache.org/jira/browse/STORM-2071?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15450483#comment-15450483
]
Jungtaek Lim commented on STORM-2071:
-
Does this occur from 1.0.1, or 1.0.x-branch (I mean 1.0.2)? If
[
https://issues.apache.org/jira/browse/STORM-2071?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15451044#comment-15451044
]
Jungtaek Lim commented on STORM-2071:
-
Just found that building master branch (2.0.0) also fails with
[
https://issues.apache.org/jira/browse/STORM-2071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jungtaek Lim updated STORM-2071:
Affects Version/s: 2.0.0
> nimbus-test test-leadership failing with Exception
>
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1656
I just pushed final version of JoinerMultiReducer for Storm SQL which
preserves fields order for each stream, which will be directly matched to
Calcite output fields.
I'll remove WIP
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1653
+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
Rajat Khandelwal created STORM-2069:
---
Summary: Wrong github url in tutorial page
Key: STORM-2069
URL: https://issues.apache.org/jira/browse/STORM-2069
Project: Apache Storm
Issue Type: Bug
Hi devs,
While implementing some features of Storm SQL on top of Trident, I realized
that there's no Trident operation which provides a way to add fields from
some Function and remove existing fields within one operation.
'each' appends the function output fields to origin input fields, and 'map'
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/1656#discussion_r76744260
--- Diff:
storm-core/src/jvm/org/apache/storm/trident/operation/impl/JoinerMultiReducer.java
---
@@ -139,21 +138,5 @@ private boolean increment(List[]
seokwoo yang created STORM-2068:
---
Summary: org.apache.storm.utils.NimbusLeaderNotFoundException
Key: STORM-2068
URL: https://issues.apache.org/jira/browse/STORM-2068
Project: Apache Storm
[
https://issues.apache.org/jira/browse/STORM-2068?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
seokwoo yang updated STORM-2068:
Description:
my storm version is apache-storm-2.0.0-SNAPSHOT
from git clone "storm github", mvn
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1656
Rebased only STORM_2057 related commits into one.
---
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-1278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15448475#comment-15448475
]
Abhishek Agarwal commented on STORM-1278:
-
FYI, I am working on this.
> port
[
https://issues.apache.org/jira/browse/STORM-1278?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Abhishek Agarwal reassigned STORM-1278:
---
Assignee: Abhishek Agarwal
> port backtype.storm.daemon.worker to java
>
[
https://issues.apache.org/jira/browse/STORM-2068?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
seokwoo yang updated STORM-2068:
Description:
my storm version is apache-storm-2.0.0-SNAPSHOT
from git clone "storm github", mvn
GitHub user shani-einav opened a pull request:
https://github.com/apache/storm/pull/1657
storm V1.0.2
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Aloomaio/incubator-storm v1.0.2
Alternatively you can review and apply these
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1656
Just addressed documentation. Ready for the review.
---
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-2068?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15448439#comment-15448439
]
Jungtaek Lim commented on STORM-2068:
-
Please post this question to user@ mailing list so that one of
[
https://issues.apache.org/jira/browse/STORM-2068?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jungtaek Lim closed STORM-2068.
---
Resolution: Won't Fix
> org.apache.storm.utils.NimbusLeaderNotFoundException
>
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1480
@sakanaou
Since STORM-2016 is merged into master and 1.x-branch, so I think we can
close this.
Could you please doing it? Thanks in advance!
---
If your project is set up for it, you
[
https://issues.apache.org/jira/browse/STORM-1881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jungtaek Lim resolved STORM-1881.
-
Resolution: Fixed
Assignee: Jungtaek Lim
Fix Version/s: 1.1.0
Github user shani-einav closed the pull request at:
https://github.com/apache/storm/pull/1657
---
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
73 matches
Mail list logo