Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add LFN in exception context in Root{Primary,Secondary}FileSequence constructor #910

Open
makortel opened this issue May 1, 2024 · 1 comment
Labels

Comments

@makortel
Copy link
Collaborator

makortel commented May 1, 2024

Motivated by the exception messages shown in cms-sw/cmssw#44878 that show file read errors while constructing PoolSource without information on what file was being read. The try-catch could be placed elsewhere though (Root{Primary,Secondary}FileSequence being our best guess now.

@makortel
Copy link
Collaborator Author

makortel commented May 1, 2024

Or perhaps all root file accesses in RootFile class would need try-catch that adds the file name in the exception message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 📋 Backlog
Development

No branches or pull requests

1 participant