fix flaky test

src/query/service/src/locks/lock_manager.rs

@@ -129,10 +129,15 @@ impl LockManager {
 
         loop {
             // List all revisions and check if the current is the minimum.
-            let reply = catalog
+            let mut rev_list = catalog
                 .list_lock_revisions(list_table_lock_req.clone())
-                .await?;
-            let rev_list = reply.into_iter().map(|(x, _)| x).collect::<Vec<_>>();
+                .await?
+                .into_iter()
+                .map(|(x, _)| x)
+                .collect::<Vec<_>>();
+            // list_lock_revisions are returned in big-endian order,
+            // we need to sort them in ascending numeric order.
+            rev_list.sort();
             let position = rev_list.iter().position(|x| *x == revision).ok_or_else(||
                 // If the current is not found in list,  it means that the current has been expired.
                 ErrorCode::TableLockExpired(format!(

src/query/service/src/sessions/query_ctx.rs

@@ -1371,11 +1371,7 @@ impl TableContext for QueryContext {
         let tbl = catalog
             .get_table(&self.get_tenant(), db_name, tbl_name)
             .await?;
-        if tbl.engine() != "FUSE" || tbl.is_read_only() {
-            return Ok(None);
-        }
-
-        if tbl.is_temp() {
+        if tbl.engine() != "FUSE" || tbl.is_read_only() || tbl.is_temp() {
             return Ok(None);
         }