[jira] [Commented] (STORM-4017) isAnyWindowsProcessAlive does not work with multiple pids

2024-01-08 Thread Scott Moore (Jira)


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

Scott Moore commented on STORM-4017:


Submitted pull-request https://github.com/apache/storm/pull/3615

> isAnyWindowsProcessAlive does not work with multiple pids
> -
>
> Key: STORM-4017
> URL: https://issues.apache.org/jira/browse/STORM-4017
> Project: Apache Storm
>  Issue Type: Bug
>  Components: storm-server
>Affects Versions: 2.3.0, 2.4.0, 2.5.0, 2.6.0
> Environment: Windows
>Reporter: Scott Moore
>Priority: Major
> Attachments: tasklist-multiple-pid-filters.png
>
>  Time Spent: 1h 10m
>  Remaining Estimate: 0h
>
> The method isAnyWindowsProcessAlive in ServerUtils.java does not work with 2 
> or more pids in the supplied pid collection
> The refactor done as part of STORM-3638 has made an incorrect assumption that 
> the 'tasklist' Windows tool can accept multiple arguments of "/fi pid eq 
> " and treat this list as an 'or' list when, in fact, the /fi arguments 
> should be considered an 'and' list.
> e.g. tasklist /fi "pid eq 123" /fi "pid eq 456" means 'give me information on 
> processes where each process has a pid of 123 and 456'
> The effect of this means that isAnyWindowsProcessAlive will always return 
> false when given 2 or more (different) pids.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (STORM-4014) Add Docker file

2024-01-08 Thread Julien Nioche (Jira)


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

Julien Nioche commented on STORM-4014:
--

NoteĀ 

The author of [https://github.com/31z4/storm-docker] has mentioned in a private 
discussion that he would be willing to contribute the code to Apache Storm.
I will move the discussion with him to a public mailing list.

> Add Docker file 
> 
>
> Key: STORM-4014
> URL: https://issues.apache.org/jira/browse/STORM-4014
> Project: Apache Storm
>  Issue Type: New Feature
>Reporter: Julien Nioche
>Priority: Major
> Fix For: 2.7.0
>
>
> See https://lists.apache.org/thread/cdcq5c9bbjd09n574sq147vcjqpt3mqm
> We want to be able to generate Docker images for Apache Storm and push them 
> to DockerHub



--
This message was sent by Atlassian Jira
(v8.20.10#820010)