On Thu, Feb 20, 2025 at 03:48:17PM +0100, Juraj Marcin wrote:
> Maybe it could be also useful to add assertions to places where locked
> BQL is assumed and the assumption is not clear enough. Then, when
> something changes, it will fail with a clear reason instead of debugging
> race conditions that might occur.

Right.  We have plenty of those already:

$ git grep "assert(bql_locked())" | wc -l

If there's some places that we find that is not clear on BQL dependency but
relies on that, we can consider adding more if that helps clarifications.

-- 
Peter Xu


Reply via email to