databendlabs / databend

๐——๐—ฎ๐˜๐—ฎ, ๐—”๐—ป๐—ฎ๐—น๐˜†๐˜๐—ถ๐—ฐ๐˜€ & ๐—”๐—œ. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.71k stars 732 forks source link

fix: table lock flaky test #16429

Closed zhyass closed 2 weeks ago

zhyass commented 2 weeks ago

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

List lock revisions are returned in big-endian order, we need to sort them in ascending numeric order.

a4074603-bd1b-404e-bfc3-9a9617ff529b 2024-09-10T11:54:05.238526+08:00  WARN 
databend_query::locks::lock_manager: lock_manager.rs:135 list_lock_revisions reply: 
[(103, LockMeta { user: "root", node: "j3MoOFeUWV7kdHDgmWxxR2", query_id: "e67d5d9a-3e7f-466b-b923-2264077ce8ae", created_on: 2024-09-10T03:54:05.195603634Z, acquired_on: Some(2024-09-10T03:54:05.204791839Z), lock_type: TABLE, extra_info: {} }), 
(106, LockMeta { user: "root", node: "j3MoOFeUWV7kdHDgmWxxR2", query_id: "a4074603-bd1b-404e-bfc3-9a9617ff529b", created_on: 2024-09-10T03:54:05.228411910Z, acquired_on: None, lock_type: TABLE, extra_info: {} }), 
(97, LockMeta { user: "root", node: "j3MoOFeUWV7kdHDgmWxxR2", query_id: "81b42e53-23b2-44b6-b215-9c45bed02c6c", created_on: 2024-09-10T03:54:05.165842784Z, acquired_on: Some(2024-09-10T03:54:05.177124526Z), lock_type: TABLE, extra_info: {} }), 
(99, LockMeta { user: "root", node: "j3MoOFeUWV7kdHDgmWxxR2", query_id: "29fe86a8-e31a-4884-b098-01dec0625614", created_on: 2024-09-10T03:54:05.173372779Z, acquired_on: None, lock_type: TABLE, extra_info: {} })]

Tests

Type of change


This change isโ€‚Reviewable