Uploaded image for project: 'Apache Storm'
  1. Apache Storm
  2. STORM-4017

isAnyWindowsProcessAlive does not work with multiple pids

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.3.0, 2.4.0, 2.5.0, 2.6.0
    • 2.6.1
    • storm-server
    • None
    • Windows

    Description

      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 <id>" 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.

      Attachments

        Issue Links

          Activity

            People

              scomo Scott Moore
              scomo Scott Moore
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 1.5h
                  1.5h