fix: remove unnecessary type conversion in MysqlWorker

[Huachao]

Which issue does this PR close?

Closes #596

Rationale for this change

What changes are included in this PR?

Remove Output to Response convension

Are there any user-facing changes?

No

How does this change test

CI

[codecov-commenter]

Codecov Report

Merging #606 (9c2f71b) into main (639c2bd) will increase coverage by 0.04%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #606      +/-   ##
==========================================
+ Coverage   65.97%   66.01%   +0.04%     
==========================================
  Files         285      285              
  Lines       44341    44358      +17     
==========================================
+ Hits        29256    29285      +29     
+ Misses      15085    15073      -12     

Impacted Files	Coverage Δ
server/src/mysql/worker.rs	0.00% <0.00%> (ø)
server/src/mysql/writer.rs	56.54% <50.66%> (+13.21%)	⬆️
wal/src/rocks_impl/manager.rs	92.53% <0.00%> (-0.19%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[jiacai2050]

@Huachao Thanks for your contribution.

Our testcases doesn't cover MySQL protocols now, would you mind add one? or you can just test by yourself, and paste what you tested here.

[Huachao]

@jiacai2050 currently I test it manually with following sql statements.

1. Create a test table

CREATE TABLE `test` (`name` string TAG, `value` double NOT NULL, `t` timestamp NOT NULL, TIMESTAMP KEY(t)) ENGINE=Analytic with (enable_ttl='false');

2. Check the empty test table output of select query

mysql> select * from test;
Query OK, 0 rows affected (0.05 sec)

3. Insert the first record

insert into test (t, name, value) VALUES (1651737067000, 'ceresdb', 100);

4. Check the test table output of select query

mysql> select * from test;
+----------------------+---------------+---------+-------+
| tsid                 | t             | name    | value |
+----------------------+---------------+---------+-------+
| 12128845460635970325 | 1651737067000 | ceresdb |   100 |
+----------------------+---------------+---------+-------+
1 row in set (0.00 sec)

5. Insert the second record

insert into test (t, name, value) VALUES (1651737068000, 'ceresdb', 200);

6. Check the test table output of select query

mysql> select * from test;
+----------------------+---------------+---------+-------+
| tsid                 | t             | name    | value |
+----------------------+---------------+---------+-------+
| 12128845460635970325 | 1651737067000 | ceresdb |   100 |
| 12128845460635970325 | 1651737068000 | ceresdb |   200 |
+----------------------+---------------+---------+-------+
2 rows in set (0.00 sec)

[jiacai2050]

The tests look good 👍

We can merge this PR after you fix column name extraction using only first RecordBatch.

[jiacai2050]

@Huachao I have push minor updates to your branch, will merge when CI pass.

[jiacai2050]

LGTM

[jiacai2050]

Thanks again 🚀