Skip to content

FIX: SRM2Storage and RequestAgentBase#705

Closed
KrzysztofCiba wants to merge 5 commits into
DIRACGrid:integrationfrom
KrzysztofCiba:8733f2453e3a71f1e0ea98194fd6814b45e54bfe
Closed

FIX: SRM2Storage and RequestAgentBase#705
KrzysztofCiba wants to merge 5 commits into
DIRACGrid:integrationfrom
KrzysztofCiba:8733f2453e3a71f1e0ea98194fd6814b45e54bfe

Conversation

@KrzysztofCiba
Copy link
Copy Markdown
Contributor

  • SRM2Storage: pipes for speed up
  • RequestAgentBase: self.log.info, not self.info!

@KrzysztofCiba
Copy link
Copy Markdown
Contributor Author

All changes in ProcessPool made here are harmless. This watchdog stuff will be present in a separate pull.

@graciani
Copy link
Copy Markdown
Contributor

It still can not be merged, and in any case the pipe approach should not be integrated if it not proven to improve the performance, KISS.

@atsareg
Copy link
Copy Markdown
Contributor

atsareg commented Apr 26, 2012

So, may be the request should be further more split to be able to incorporate the ProcessPool fixes

@KrzysztofCiba
Copy link
Copy Markdown
Contributor Author

@KrzysztofCiba
Copy link
Copy Markdown
Contributor Author

Closing, SRM2Storage goes to new branch.

fstagni pushed a commit to Robin-Van-de-Merghel/DIRAC that referenced this pull request Dec 8, 2025
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.14.4 → v0.14.6](astral-sh/ruff-pre-commit@v0.14.4...v0.14.6)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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