From 24dac33197590dfe3fa229ff0f1a0d3b9bc5c586 Mon Sep 17 00:00:00 2001
From: zjregee <zjregee@gmail.com>
Date: Fri, 6 Sep 2024 18:58:28 +0800
Subject: [PATCH] modify readme

---
 README.md                         | 2 +-
 sqllogictest-engines/src/mysql.rs | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/README.md b/README.md
index 94959e0..58e0ae9 100644
--- a/README.md
+++ b/README.md
@@ -44,7 +44,7 @@ You can find more options in `sqllogictest --help` .
 
 > **Note**
 >
-> Currently only postgres is supported in the CLI tool.
+> Currently only postgres and mysql are supported in the CLI tool.
 
 ## `.slt` Test File Format Cookbook
 
diff --git a/sqllogictest-engines/src/mysql.rs b/sqllogictest-engines/src/mysql.rs
index 84245d7..3e4d121 100644
--- a/sqllogictest-engines/src/mysql.rs
+++ b/sqllogictest-engines/src/mysql.rs
@@ -42,9 +42,10 @@ impl sqllogictest::AsyncDB for MySql {
                 let value_str = match value {
                     Value::Bytes(bytes) => match String::from_utf8(bytes) {
                         Ok(x) => x,
-                        Err(_) => "NULL".to_string(),
+                        Err(_) => unreachable!(),
                     },
-                    _ => "NULL".to_string(),
+                    Value::NULL => "NULL".to_string(),
+                    _ => unreachable!(),
                 };
                 if value_str.is_empty() {
                     row_vec.push("(empty)".to_string());