microsoft / ConversationLearner-SDK

An SDK for ConversationLearner. REPOSITORY MOVED
https://github.com/microsoft/conversationlearner
MIT License
26 stars 9 forks source link

fix: bug 2417 multi-conversation message queuing #522

Closed LarsLiden closed 4 years ago

LarsLiden commented 4 years ago

When we refactored the memory we switched to using a conversationId scoped mutex but were still using a shared queue which caused multi-conversations to run into issues. Fixed the bug and also took opportunity to clean it up a bit and add tests.

Note queue is still in-memory so won't work for multi-host bots. See ADO 2412

mattmazzola commented 4 years ago

:tada: This PR is included in version 0.344.3 :tada:

The release is available on:

Your semantic-release bot :package::rocket: