Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: DiamondLightSource/python-procrunner
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
Loading
...
head repository: DiamondLightSource/python-procrunner
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: nbsr-underrun
Choose a head ref
Loading
Checking mergeability… Don’t worry, you can still create the pull request.
  • 1 commit
  • 1 file changed
  • 1 contributor

Commits on Mar 3, 2019

  1. Limit time spent waiting on stream reader threads

    There may be cases where the stream reader can not finish even though
    the subprocess has already terminated. For example the subprocess can
    pass the stdout/stderr handles on to a daemon process which will keep
    them open. In those cases we need to stop reading at some point to
    continue the normal control flow.
    
    Proposed solution: wait for 1 second for the stream reader to catch up,
    and if the stream is still held open or the stream reader is still
    running ignore any further output and continue.
    
    Downside: this is essentially a memory leak as the stream reader thread
    may still linger on. To limit the potential damage add a check for the
    ._closed property in the main stream reader loop and terminate there.
    
    Might resolve #19.
    Anthchirp committed Mar 3, 2019
    Configuration menu
    Copy the full SHA
    bb71233 View commit details
    Browse the repository at this point in the history
Loading
Morty Proxy This is a proxified and sanitized view of the page, visit original site.