Page MenuHomeSoftware Heritage

listener: Release the db object after using it
ClosedPublic

Authored by olasd on Tue, May 7, 6:17 PM.

Details

Summary

This is the contract that get_db/put_db is supposed to conform to.

Test Plan

tox is still happy; we can process more than ten pending messages at once

Diff Detail

Repository
rDSCH Scheduling utilities
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

olasd created this revision.Tue, May 7, 6:17 PM

It should probably be in a finally block.

olasd updated this revision to Diff 4777.Fri, May 10, 2:38 PM

Less improper database exception handling

vlorentz accepted this revision.Fri, May 10, 3:52 PM
This revision is now accepted and ready to land.Fri, May 10, 3:52 PM
olasd updated this revision to Diff 4779.Fri, May 10, 3:52 PM

Rrrrrebase

This revision was automatically updated to reflect the committed changes.