Skip to content

fix: Guard against crashes stemming from pdfium in threaded environments#3172

Closed
cau-git wants to merge 6 commits into
mainfrom
cau/pdfium-race-fix
Closed

fix: Guard against crashes stemming from pdfium in threaded environments#3172
cau-git wants to merge 6 commits into
mainfrom
cau/pdfium-race-fix

Conversation

@cau-git
Copy link
Copy Markdown
Member

@cau-git cau-git commented Mar 23, 2026

Checklist:

  • Documentation has been updated, if necessary.
  • Examples have been added, if necessary.
  • Tests have been added, if necessary.

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Mar 23, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

Signed-off-by: Christoph Auer <cau@zurich.ibm.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 23, 2026

DCO Check Passed

Thanks @cau-git, all your commits are properly signed off. 🎉

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 23, 2026

Codecov Report

❌ Patch coverage is 88.42105% with 22 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
docling/pipeline/standard_pdf_pipeline.py 71.21% 19 Missing ⚠️
docling/backend/managed_pdfium_backend.py 95.58% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

cau-git added 2 commits March 24, 2026 10:34
Signed-off-by: Christoph Auer <cau@zurich.ibm.com>
Signed-off-by: Christoph Auer <cau@zurich.ibm.com>
@cau-git
Copy link
Copy Markdown
Member Author

cau-git commented Mar 24, 2026

Closing in favour of #3180

@cau-git cau-git closed this Mar 24, 2026
@dolfim-ibm dolfim-ibm deleted the cau/pdfium-race-fix branch May 6, 2026 11:30
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.

1 participant