Skip to content

FIX: ProcessPool with LockRing#683

Merged
atsareg merged 5 commits into
DIRACGrid:integrationfrom
KrzysztofCiba:FIX-DMS-refactoring
Apr 23, 2012
Merged

FIX: ProcessPool with LockRing#683
atsareg merged 5 commits into
DIRACGrid:integrationfrom
KrzysztofCiba:FIX-DMS-refactoring

Conversation

@KrzysztofCiba
Copy link
Copy Markdown
Contributor

Now in a proper place.

Also decrease verbosity in TA and RAB.

@atsareg
Copy link
Copy Markdown
Contributor

atsareg commented Apr 19, 2012

I see that you have suppressed the execution order check. This is because you rely on the updated functionality in the RequestDBMySQL which returns requests with the write execution order ? This functionality will be available in v6r4. So, this pull request is not for v6r3p6 patch.

@graciani
Copy link
Copy Markdown
Contributor

and:

      if task.isBullet():
        break
      self.__working.value = 1
      try:
        task.process()
      if task.hasCallback() or task.usePoolCallbacks():
        self.__resultsQueue.put( task, block = True )
      finally:
        self.__working.value = 0

should be:

      if task.isBullet():
        break
      self.__working.value = 1
      try:
        task.process()
        if task.hasCallback() or task.usePoolCallbacks():
          self.__resultsQueue.put( task, block = True )
      finally:
        self.__working.value = 0

@KrzysztofCiba
Copy link
Copy Markdown
Contributor Author

Fixed. Thanks for spotting.

@KrzysztofCiba
Copy link
Copy Markdown
Contributor Author

New locks put in ProcessPool:

  • PP.__spawnWorkignProcess
  • PP.__killWorkingProcess
  • PP.queueTask

…_workersDict for better tracking of spawned WorkingProcesses
@KrzysztofCiba
Copy link
Copy Markdown
Contributor Author

This will allow to track 'stuck' processes at the ned of MxCycles for all agent. Previously all workers were hold in a list, which was 'regenerated' in a few places, now all created workers are kept in a dict with worker.pid as a key.

atsareg added a commit that referenced this pull request Apr 23, 2012
@atsareg atsareg merged commit eafc04e into DIRACGrid:integration Apr 23, 2012
fstagni pushed a commit to Robin-Van-de-Merghel/DIRAC that referenced this pull request Dec 8, 2025
…Grid#683)

- Trigger real deployments on version tags (v*)
- Remove obsolete manual release creation logic and script
- Update DIRACCommon dependency to stable version (9.0.0)
- Simplify version extraction to use tag references directly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants