[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 request into a Git repository by running:

$ git pull https://github.com/danielsd/storm STORM-2791-1.x

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/storm/pull/2392.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2392


commit af1fd216a623e142f18248cd4d40ca1fbc19a9c4
Author: Daniel Dara 
Date:   2017-10-28T02:32:29Z

Deprecated FixedTupleSpout constructor with String. Added constructor with 
Fields so more than one output field can be specified.




---


[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 repository by running:

$ git pull https://github.com/danielsd/storm STORM-2791

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/storm/pull/2391.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2391


commit 3fde12cce96dcc38c8f98a181ea76e3a4745425a
Author: Daniel Dara 
Date:   2017-10-28T02:11:02Z

Deprecated FixedTupleSpout constructor with String. Added constructor with 
Fields so more than one output field can be specified.




---


[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 https://github.com/govind-menon/storm YSTORM-2727

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/storm/pull/2385.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2385


commit facf515b121c80eed6c02d74104e323cfb9e4a1d
Author: Govind Menon 
Date:   2017-09-07T18:50:05Z

YSTORM-2725: Generic Resource Scheduling - initial config changes and 
TopologyBuilder API

commit 4337d44b4f95da34f2b1d0a4160353ab81c0fab0
Author: Govind Menon 
Date:   2017-10-23T15:31:08Z

YSTORM-2727: Generic Resource Aware Scheduling(GRAS) - metadata, scheduling 
strategies and configuration enhancements

commit 8e867fcf9183e8476658bff7119d9de8545e9aed
Author: Govind Menon 
Date:   2017-10-23T15:31:08Z

YSTORM-2727: Generic Resource Aware Scheduling(GRAS) - metadata, scheduling 
strategies and configuration enhancements




---


[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

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/storm/pull/2390.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2390


commit e7d9881c1876494b179e06c7c3ee64c606590343
Author: Kishor Patil 
Date:   2017-10-26T21:59:35Z

Add nimbus admins 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


---