GitHub user vesense opened a pull request:
https://github.com/apache/storm/pull/967
Remove repeated semicolon
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vesense/storm patch-14
Alternatively you can review and apply these ch
GitHub user vesense opened a pull request:
https://github.com/apache/storm/pull/968
Add 'public' for EventImpl class
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vesense/storm patch-15
Alternatively you can review and apply t
Xin Wang created STORM-1411:
---
Summary: make EvictionPolicy type safety
Key: STORM-1411
URL: https://issues.apache.org/jira/browse/STORM-1411
Project: Apache Storm
Issue Type: Bug
Report
GitHub user vesense opened a pull request:
https://github.com/apache/storm/pull/969
[STORM-1411] make windowing EvictionPolicy type safety
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vesense/storm patch-16
Alternatively you
[
https://issues.apache.org/jira/browse/STORM-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1411:
Summary: make windowing EvictionPolicy type safety (was: make
EvictionPolicy type safety)
> make windowing
[
https://issues.apache.org/jira/browse/STORM-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069385#comment-15069385
]
ASF GitHub Bot commented on STORM-1411:
---
GitHub user vesense opened a pull request:
Xin Wang created STORM-1412:
---
Summary: Null check should be done in the first place
Key: STORM-1412
URL: https://issues.apache.org/jira/browse/STORM-1412
Project: Apache Storm
Issue Type: Bug
[
https://issues.apache.org/jira/browse/STORM-1412?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1412:
Description:
Null check should be done in the first place for class User, otherwise we may
get a NPE.
befor
GitHub user vesense opened a pull request:
https://github.com/apache/storm/pull/970
[STORM-1412] null check should be done in the first place
https://issues.apache.org/jira/browse/STORM-1412
You can merge this pull request into a Git repository by running:
$ git pull https://gi
[
https://issues.apache.org/jira/browse/STORM-1412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069404#comment-15069404
]
ASF GitHub Bot commented on STORM-1412:
---
GitHub user vesense opened a pull request:
[
https://issues.apache.org/jira/browse/STORM-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1411:
Component/s: storm-core
> make windowing EvictionPolicy type safety
> --
Github user vesense commented on the pull request:
https://github.com/apache/storm/pull/969#issuecomment-166838975
The Travis CI failed build seems unrelated.
---
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 projec
[
https://issues.apache.org/jira/browse/STORM-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069413#comment-15069413
]
ASF GitHub Bot commented on STORM-1411:
---
Github user vesense commented on the pull r
[
https://issues.apache.org/jira/browse/STORM-1139?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
hima reopened STORM-1139:
-
> Issues regarding storm-postgresql interface
> ---
>
> Key: S
[
https://issues.apache.org/jira/browse/STORM-1139?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15066101#comment-15066101
]
hima edited comment on STORM-1139 at 12/23/15 9:15 AM:
---
hai..
[
https://issues.apache.org/jira/browse/STORM-1139?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069419#comment-15069419
]
hima commented on STORM-1139:
-
hai..
Using storm bolts I want to insert the data into the Pos
Xin Wang created STORM-1413:
---
Summary: remove unused variables for localizer test
Key: STORM-1413
URL: https://issues.apache.org/jira/browse/STORM-1413
Project: Apache Storm
Issue Type: Bug
GitHub user vesense opened a pull request:
https://github.com/apache/storm/pull/971
[STORM-1413] remove unused variables for localizer test
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vesense/storm patch-18
Alternatively you
[
https://issues.apache.org/jira/browse/STORM-1413?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1413:
Component/s: storm-core
> remove unused variables for localizer test
> -
[
https://issues.apache.org/jira/browse/STORM-1413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069426#comment-15069426
]
ASF GitHub Bot commented on STORM-1413:
---
GitHub user vesense opened a pull request:
GitHub user vesense opened a pull request:
https://github.com/apache/storm/pull/972
[STORM-1414] refactor multilang JsonSerializer
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vesense/storm patch-19
Alternatively you can revi
Xin Wang created STORM-1414:
---
Summary: refactor multilang JsonSerializer
Key: STORM-1414
URL: https://issues.apache.org/jira/browse/STORM-1414
Project: Apache Storm
Issue Type: Improvement
[
https://issues.apache.org/jira/browse/STORM-1414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069505#comment-15069505
]
ASF GitHub Bot commented on STORM-1414:
---
GitHub user vesense opened a pull request:
Xin Wang created STORM-1415:
---
Summary: refactor trident map StateUpdater
Key: STORM-1415
URL: https://issues.apache.org/jira/browse/STORM-1415
Project: Apache Storm
Issue Type: Improvement
[
https://issues.apache.org/jira/browse/STORM-1415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069523#comment-15069523
]
ASF GitHub Bot commented on STORM-1415:
---
GitHub user vesense opened a pull request:
GitHub user vesense opened a pull request:
https://github.com/apache/storm/pull/973
[STORM-1415] refactor trident map StateUpdater
https://issues.apache.org/jira/browse/STORM-1415
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ve
Arun Mahadevan created STORM-1416:
-
Summary: Add documentation for windowing state store functionality
Key: STORM-1416
URL: https://issues.apache.org/jira/browse/STORM-1416
Project: Apache Storm
Github user arunmahadevan commented on the pull request:
https://github.com/apache/storm/pull/939#issuecomment-166891539
@Parth-Brahmbhatt @harshach refactored the state handling code to make it
cleaner and addressed other review comments. Also added a followup task for
documentation
[
https://issues.apache.org/jira/browse/STORM-1175?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069611#comment-15069611
]
ASF GitHub Bot commented on STORM-1175:
---
Github user arunmahadevan commented on the
Github user vesense commented on the pull request:
https://github.com/apache/storm/pull/965#issuecomment-166898038
LGTM +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
[
https://issues.apache.org/jira/browse/STORM-1408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069628#comment-15069628
]
ASF GitHub Bot commented on STORM-1408:
---
Github user vesense commented on the pull r
Github user vesense commented on the pull request:
https://github.com/apache/storm/pull/964#issuecomment-166898153
LGTM +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
[
https://issues.apache.org/jira/browse/STORM-1409?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069627#comment-15069627
]
ASF GitHub Bot commented on STORM-1409:
---
Github user vesense commented on the pull r
[
https://issues.apache.org/jira/browse/STORM-1211?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Satish Duggana resolved STORM-1211.
---
Resolution: Fixed
> Add trident state and query support for cassandra connector
>
Satish Duggana created STORM-1417:
-
Summary: CoordType breaks equlas/hashCode contract
Key: STORM-1417
URL: https://issues.apache.org/jira/browse/STORM-1417
Project: Apache Storm
Issue Type:
GitHub user vesense opened a pull request:
https://github.com/apache/storm/pull/974
remove unused local variable
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vesense/storm patch-21
Alternatively you can review and apply these
GitHub user satishd opened a pull request:
https://github.com/apache/storm/pull/975
STORM-1417: fixed equals/hashCode contract in CoordType
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/satishd/storm STORM-1417
Alternatively y
[
https://issues.apache.org/jira/browse/STORM-1417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069658#comment-15069658
]
ASF GitHub Bot commented on STORM-1417:
---
GitHub user satishd opened a pull request:
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/974#issuecomment-166936730
+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
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/971#issuecomment-166936941
@vesense thanks for these fixes but please send these PRs under 1 jira.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
[
https://issues.apache.org/jira/browse/STORM-1413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069835#comment-15069835
]
ASF GitHub Bot commented on STORM-1413:
---
Github user harshach commented on the pull
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/975#issuecomment-166940418
nice catch @satishd. +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 thi
[
https://issues.apache.org/jira/browse/STORM-1417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069848#comment-15069848
]
ASF GitHub Bot commented on STORM-1417:
---
Github user harshach commented on the pull
[
https://issues.apache.org/jira/browse/STORM-1405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069851#comment-15069851
]
ASF GitHub Bot commented on STORM-1405:
---
Github user harshach commented on the pull
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/960#issuecomment-166941003
+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
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/961#issuecomment-166941112
+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
[
https://issues.apache.org/jira/browse/STORM-1407?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069852#comment-15069852
]
ASF GitHub Bot commented on STORM-1407:
---
Github user harshach commented on the pull
Github user arunmahadevan commented on the pull request:
https://github.com/apache/storm/pull/969#issuecomment-166941813
@vesense can you submit it in a single commit?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
[
https://issues.apache.org/jira/browse/STORM-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069856#comment-15069856
]
ASF GitHub Bot commented on STORM-1411:
---
Github user arunmahadevan commented on the
Github user arunmahadevan commented on the pull request:
https://github.com/apache/storm/pull/968#issuecomment-166943815
@vesense can you submit it as a part of JIRA? You could club this under
STORM-1411 that you already raised and change the Jira title accordingly.
---
If your proje
[
https://issues.apache.org/jira/browse/STORM-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15069864#comment-15069864
]
ASF GitHub Bot commented on STORM-1411:
---
Github user arunmahadevan commented on the
Github user harshach commented on a diff in the pull request:
https://github.com/apache/storm/pull/939#discussion_r48368972
--- Diff: storm-core/src/jvm/backtype/storm/spout/CheckpointSpout.java ---
@@ -0,0 +1,280 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF)
[
https://issues.apache.org/jira/browse/STORM-1175?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15070019#comment-15070019
]
ASF GitHub Bot commented on STORM-1175:
---
Github user harshach commented on a diff in
[
https://issues.apache.org/jira/browse/STORM-1175?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15070020#comment-15070020
]
ASF GitHub Bot commented on STORM-1175:
---
Github user harshach commented on the pull
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/939#issuecomment-166966801
+1 as well. minor nit pick on the default value.
---
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
[
https://issues.apache.org/jira/browse/STORM-1175?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15070094#comment-15070094
]
ASF GitHub Bot commented on STORM-1175:
---
Github user arunmahadevan commented on the
Github user arunmahadevan commented on the pull request:
https://github.com/apache/storm/pull/939#issuecomment-166977965
@harshach while addressing review comments I had added the config in
defaults.yaml. I further refactored it to remove the constant and made sure
that the value is n
Github user zhuoliu commented on the pull request:
https://github.com/apache/storm/pull/974#issuecomment-166984451
+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
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/962
---
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 enabl
Github user harshach commented on a diff in the pull request:
https://github.com/apache/storm/pull/936#discussion_r48379489
--- Diff:
external/storm-hdfs/src/main/java/org/apache/storm/hdfs/spout/DirLock.java ---
@@ -0,0 +1,126 @@
+/**
+ * Licensed to the Apache Software Fo
[
https://issues.apache.org/jira/browse/STORM-1199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15070194#comment-15070194
]
ASF GitHub Bot commented on STORM-1199:
---
Github user harshach commented on a diff in
Github user vesense commented on the pull request:
https://github.com/apache/storm/pull/971#issuecomment-167018639
@harshach Thanks. I will fix it.
---
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
[
https://issues.apache.org/jira/browse/STORM-1413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15070369#comment-15070369
]
ASF GitHub Bot commented on STORM-1413:
---
Github user vesense commented on the pull r
[
https://issues.apache.org/jira/browse/STORM-1413?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1413:
Summary: remove unused variables for some tests (was: remove unused
variables for localizer test)
> remove
[
https://issues.apache.org/jira/browse/STORM-1413?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1413:
Description:
1.backtype/storm/blobstore/BlobStoreTest.java
2.backtype/storm/localizer/LocalizerTest.java
>
[
https://issues.apache.org/jira/browse/STORM-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1411:
Summary: some fixes for storm-windowing (was: make windowing
EvictionPolicy type safety)
> some fixes for
[
https://issues.apache.org/jira/browse/STORM-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1411:
Description:
Changes are following:
1.Make windowing EvictionPolicy type safety
2.Add 'public' for windowing
Github user vesense commented on the pull request:
https://github.com/apache/storm/pull/968#issuecomment-167022720
@arunmahadevan Thanks. I will fix that.
---
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 do
[
https://issues.apache.org/jira/browse/STORM-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1411:
Description:
Changes are the following:
1.Make windowing EvictionPolicy type safety
2.Add 'public' for windo
[
https://issues.apache.org/jira/browse/STORM-1414?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1414:
Description:
Changes are the following:
1.Add a generated serialVersionUID & default charset 'UTF-8'
2.Use B
[
https://issues.apache.org/jira/browse/STORM-1415?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1415:
Description:
Changes are the following:
1.Add a generated serialVersionUID
2.Remove unused variables 'groups
[
https://issues.apache.org/jira/browse/STORM-1413?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Xin Wang updated STORM-1413:
Description:
Fixed test classes are the following:
1.backtype/storm/blobstore/BlobStoreTest.java
2.backtype/
Github user 2new commented on the pull request:
https://github.com/apache/storm/pull/972#issuecomment-167039898
+1 LGTM
---
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 an
[
https://issues.apache.org/jira/browse/STORM-1414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15070574#comment-15070574
]
ASF GitHub Bot commented on STORM-1414:
---
Github user 2new commented on the pull requ
I want to import a project that run in two modes "local and production"
mode
want to get what is mean by production mode is that mean in cluster or
distributed mode ?
and if that right . are the configurations in storm.yaml is like that or
not ?
storm.zookeeper.servers:
- "ipaddress"
nim
[
https://issues.apache.org/jira/browse/STORM-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15070694#comment-15070694
]
ASF GitHub Bot commented on STORM-1411:
---
Github user vesense commented on the pull r
Github user vesense commented on the pull request:
https://github.com/apache/storm/pull/969#issuecomment-167060100
@arunmahadevan Thanks. I have rebased the PR.
---
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 proj
77 matches
Mail list logo
