[jira] [Commented] (STORM-2796) Flux: Provide means for invoking static factory methods

2017-11-03 Thread Roshan Naik (JIRA)

[ 
https://issues.apache.org/jira/browse/STORM-2796?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16238504#comment-16238504
 ] 

Roshan Naik commented on STORM-2796:


Yaml looks fine. Would be nice to make sure that  its works with this type of 
method signatures as well:

{code}
public static MyComponent newInstance(Integer x, SomeObject... variadic)
{code}

Basically be able to pass a mix of objects and arrays/variadics as args. I 
think the ability to pass array args is done via an undocumented feature called 
`reflist`. Just want to make sure it it supported for factory methods as well.

Which reminds me that it would be good to update the docs on 'reflist'  while 
updating the docs about factor methods.


> Flux: Provide means for invoking static factory methods
> ---
>
> Key: STORM-2796
> URL: https://issues.apache.org/jira/browse/STORM-2796
> Project: Apache Storm
>  Issue Type: Improvement
>  Components: Flux
>Affects Versions: 2.0.0, 1.1.1, 1.2.0, 1.0.6
>Reporter: P. Taylor Goetz
>Assignee: P. Taylor Goetz
>Priority: Normal
>
> Provide a means to invoke static factory methods for flux components. E.g:
> Java signature:
> {code}
> public static MyComponent newInstance(String... params)
> {code}
> Yaml:
> {code}
> className: "org.apache.storm.flux.test.MyComponent"
> factory: "newInstance"
> factoryArgs: ["a", "b", "c"]
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (STORM-2796) Flux: Provide means for invoking static factory methods

2017-11-03 Thread P. Taylor Goetz (JIRA)

 [ 
https://issues.apache.org/jira/browse/STORM-2796?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

P. Taylor Goetz updated STORM-2796:
---
Description: 
Provide a means to invoke static factory methods for flux components. E.g:

Java signature:
{code}
public static MyComponent newInstance(String... params)
{code}

Yaml:

{code}
className: "org.apache.storm.flux.test.MyComponent"
factory: "newInstance"
factoryArgs: ["a", "b", "c"]
{code}

  was:
Provide a means to invoke static factory methods for flux components. E.g:

Java signature:
{code}
public static MyComponent newInstance(String... params)
{code}

Yaml:

{code}
className: "org.apache.storm.flux.test.MyComponent"
factory: "factory"
factoryArgs: ["a", "b", "c"]
{code}


> Flux: Provide means for invoking static factory methods
> ---
>
> Key: STORM-2796
> URL: https://issues.apache.org/jira/browse/STORM-2796
> Project: Apache Storm
>  Issue Type: Improvement
>  Components: Flux
>Affects Versions: 2.0.0, 1.1.1, 1.2.0, 1.0.6
>Reporter: P. Taylor Goetz
>Assignee: P. Taylor Goetz
>Priority: Normal
>
> Provide a means to invoke static factory methods for flux components. E.g:
> Java signature:
> {code}
> public static MyComponent newInstance(String... params)
> {code}
> Yaml:
> {code}
> className: "org.apache.storm.flux.test.MyComponent"
> factory: "newInstance"
> factoryArgs: ["a", "b", "c"]
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Created] (STORM-2796) Flux: Provide means for invoking static factory methods

2017-11-03 Thread P. Taylor Goetz (JIRA)
P. Taylor Goetz created STORM-2796:
--

 Summary: Flux: Provide means for invoking static factory methods
 Key: STORM-2796
 URL: https://issues.apache.org/jira/browse/STORM-2796
 Project: Apache Storm
  Issue Type: Improvement
  Components: Flux
Affects Versions: 2.0.0, 1.1.1, 1.2.0, 1.0.6
Reporter: P. Taylor Goetz
Assignee: P. Taylor Goetz
Priority: Normal


Provide a means to invoke static factory methods for flux components. E.g:

Java signature:
{code}
public static MyComponent newInstance(String... params)
{code}

Yaml:

{code}
className: "org.apache.storm.flux.test.MyComponent"
factory: "factory"
factoryArgs: ["a", "b", "c"]
{code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Resolved] (STORM-2795) Race in downloading resources can cause failure

2017-11-03 Thread Robert Joseph Evans (JIRA)

 [ 
https://issues.apache.org/jira/browse/STORM-2795?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Joseph Evans resolved STORM-2795.

   Resolution: Fixed
Fix Version/s: 2.0.0

Merged this into master

> Race in downloading resources can cause failure
> ---
>
> Key: STORM-2795
> URL: https://issues.apache.org/jira/browse/STORM-2795
> Project: Apache Storm
>  Issue Type: Bug
>  Components: storm-server
>Affects Versions: 2.0.0
>Reporter: Robert Joseph Evans
>Assignee: Robert Joseph Evans
>Priority: Major
>  Labels: pull-request-available
> Fix For: 2.0.0
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> Recently had a failure/hang in the async localizer test.  Turns out that 
> there is a race when downloading dependencies and there is a race in trying 
> to create the parent directory.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (STORM-2792) Clean up RAS and remove possible loops

2017-11-03 Thread ASF GitHub Bot (JIRA)

 [ 
https://issues.apache.org/jira/browse/STORM-2792?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

ASF GitHub Bot updated STORM-2792:
--
Labels: pull-request-available  (was: )

> Clean up RAS and remove possible loops
> --
>
> Key: STORM-2792
> URL: https://issues.apache.org/jira/browse/STORM-2792
> Project: Apache Storm
>  Issue Type: Improvement
>Reporter: Robert Joseph Evans
>Assignee: Robert Joseph Evans
>Priority: Major
>  Labels: pull-request-available
>
> The code for RAS is rather complex, and we have found that if there is a 
> mismatch between the priority strategy and the eviction strategy that it can 
> result in infinite loops.  To make this simpler there really should just be 
> one strategy to prioritize all topologies.  Scheduling happens for the 
> highest priority topologies and eviction happens for the lowest priority 
> ones. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)