mirror of
https://github.com/trustgraph-ai/trustgraph.git
synced 2026-05-10 07:42:38 +02:00
Fix publisher resource leak in librarian submit_document (#883)
Wrap pub.start()/pub.send() in try/finally to guarantee pub.stop() is called on error. Remove unnecessary asyncio.sleep(1) kludge.
This commit is contained in:
parent
1ffae12559
commit
f9d6606423
1 changed files with 5 additions and 8 deletions
|
|
@ -418,13 +418,10 @@ class Processor(AsyncProcessor):
|
||||||
self.pubsub, q, schema=schema
|
self.pubsub, q, schema=schema
|
||||||
)
|
)
|
||||||
|
|
||||||
|
try:
|
||||||
await pub.start()
|
await pub.start()
|
||||||
|
|
||||||
# FIXME: Time wait kludge?
|
|
||||||
await asyncio.sleep(1)
|
|
||||||
|
|
||||||
await pub.send(None, doc)
|
await pub.send(None, doc)
|
||||||
|
finally:
|
||||||
await pub.stop()
|
await pub.stop()
|
||||||
|
|
||||||
logger.debug("Document submitted")
|
logger.debug("Document submitted")
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue