tarantool / crud / 29415116647

88%

Build Type

push

github

Committed by ita-sammann

Commit Message

fix: handle nil bucket_id in storage get/update/delete operations

Starting from crud 1.7.0, storage asserts if `opts.bucket_id`
is nil during `get`, `update`, or `delete` operations.
Since older routers (< 1.7.0) do not pass `bucket_id`, this
causes protocol incompatibility.

This commit adds a nil check to skip bucket ref when `bucket_id`
is missing, restoring backward compatibility with old routers.
It also introduces a rate-limited warning and the
`tnt_crud_storage_nil_bucket_id_compat_total` metric to track
this state.

Coverage Stats

55 of 62 new or added lines in 7 files covered. (88.71%)

2 existing lines in 1 file now uncovered.

5406 of 6123 relevant lines covered (88.29%)

12220.28 hits per line