[GitHub] storm pull request #2392: STORM-2791: Add Fields constructor to FixedTupleSp...

2017-10-27 Thread danielsd
GitHub user danielsd opened a pull request: https://github.com/apache/storm/pull/2392 STORM-2791: Add Fields constructor to FixedTupleSpout for multiple output field support. (1.x) [JIRA - STORM 2791](https://issues.apache.org/jira/browse/STORM-2791) You can merge this pull

[GitHub] storm pull request #2391: STORM-2791 - Add Fields constructor to FixedTupleS...

2017-10-27 Thread danielsd
GitHub user danielsd opened a pull request: https://github.com/apache/storm/pull/2391 STORM-2791 - Add Fields constructor to FixedTupleSpout for multiple output field support. Deprecated existing constructor that accepts a `String`. You can merge this pull request into a Git

[GitHub] storm issue #2387: STORM-2787: storm-kafka-client KafkaSpout method onPartit...

2017-10-27 Thread hmcl
Github user hmcl commented on the issue: https://github.com/apache/storm/pull/2387 @srdo @HeartSaVioR I have incorporated the code review changes of the depending patch. It should be good to merge. Thanks. ---

[GitHub] storm issue #2380: STORM-2781: Refactor storm-kafka-client KafkaSpout Proces...

2017-10-27 Thread hmcl
Github user hmcl commented on the issue: https://github.com/apache/storm/pull/2380 I have squashed the commits and addressed the early return issue. ---

[GitHub] storm pull request #2385: STORM-2727: Generic Resource Aware Scheduling

2017-10-27 Thread govind-menon
GitHub user govind-menon reopened a pull request: https://github.com/apache/storm/pull/2385 STORM-2727: Generic Resource Aware Scheduling Remaining 1. Add Tests 2. Do more manual testing You can merge this pull request into a Git repository by running: $ git pull

[GitHub] storm pull request #2385: STORM-2727: Generic Resource Aware Scheduling

2017-10-27 Thread govind-menon
Github user govind-menon closed the pull request at: https://github.com/apache/storm/pull/2385 ---

[GitHub] storm issue #2390: [STORM-2790] Add nimbus admins groups

2017-10-27 Thread Ethanlm
Github user Ethanlm commented on the issue: https://github.com/apache/storm/pull/2390 Looks like there is some checkstyle violatiom issues here. ---

[GitHub] storm issue #2386: [STORM-1927] upgrade to jetty 9

2017-10-27 Thread Ethanlm
Github user Ethanlm commented on the issue: https://github.com/apache/storm/pull/2386 I fixed the bugs and tested it. It should be good now. ---

[GitHub] storm pull request #2390: [STORM-2790] Add nimbus admins groups

2017-10-27 Thread kishorvpatil
GitHub user kishorvpatil opened a pull request: https://github.com/apache/storm/pull/2390 [STORM-2790] Add nimbus admins groups You can merge this pull request into a Git repository by running: $ git pull https://github.com/kishorvpatil/incubator-storm add-admin-groups

[GitHub] storm issue #2379: STORM-2782 - refactor partial key grouping to make it mor...

2017-10-27 Thread kevpeek
Github user kevpeek commented on the issue: https://github.com/apache/storm/pull/2379 current performance is between 3% and 12% slower than the old version. @revans2 Would you elaborate on what you meant by, "pre-allocate the possible return values as lists?" Thanks.

[GitHub] storm issue #2365: [STORM-2773]If a drpcserver node in cluster is down,drpc ...

2017-10-27 Thread liu-zhaokun
Github user liu-zhaokun commented on the issue: https://github.com/apache/storm/pull/2365 @revans2 Do you mean there no need to reconnect when I create a DRPCInvocationClient instance? ---

[GitHub] storm pull request #2381: STORM-2784: storm-kafka-client KafkaTupleListener ...

2017-10-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2381 ---