Skip to content

Conversation

@aylei
Copy link
Collaborator

@aylei aylei commented Aug 30, 2025

Tested (run the relevant ones):

  • Code formatting: install pre-commit (auto-check on commit) or bash format.sh
  • Any manual or new tests for this PR (please specify below)
  • All smoke tests: /smoke-test (CI) or pytest tests/test_smoke.py (local)
  • Relevant individual tests: /smoke-test -k test_name (CI) or pytest tests/test_smoke.py::test_name (local)
  • Backward compatibility: /quicktest-core (CI) or pytest tests/smoke_tests/test_backward_compat.py (local)

Signed-off-by: Aylei <rayingecho@gmail.com>
@aylei
Copy link
Collaborator Author

aylei commented Aug 30, 2025

/quicktest-core

Copy link
Collaborator

@Michaelvll Michaelvll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @aylei! Let's investigate a bit more for this unexpected aiosqlite behavior #6962

@aylei aylei merged commit 87bdb38 into master Aug 30, 2025
17 checks passed
@aylei aylei deleted the revert-aio-sqlite branch August 30, 2025 04:36
aylei added a commit that referenced this pull request Aug 30, 2025
aylei added a commit that referenced this pull request Sep 1, 2025
* Revert "Revert [Request DB asyncio support] (#6963)"

This reverts commit 87bdb38.

* Fix race condition and add UT

Signed-off-by: Aylei <rayingecho@gmail.com>

* Fix UT

Signed-off-by: Aylei <rayingecho@gmail.com>

* Merge master and enable UT

Signed-off-by: Aylei <rayingecho@gmail.com>

---------

Signed-off-by: Aylei <rayingecho@gmail.com>
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.

3 participants