Print back the # of rows affected (ins, upd, del) with timings

[mamcx]

Description of Changes

Closes #2430.

API and ABI breaking changes

It change the returning json for sql route (spacetimedb::json::client_api::StmtResultJson), with added field for stats.

Expected complexity level and risk

1

Testing

• Added test that confirm the number of stats for ins, del, upd
• Added unit test for the output of the sql cli with and without stats
• Manual testing of the cli output using the module keynote-benchmarks

🪐quickstart>delete from position;
(0 rows) [deleted: 1000000, server: 3.71s]
Roundtrip time: 3.72s

🪐quickstart>select * from position;
 id | x | y | z
----+---+---+---
(0 rows) [server: 1.32ms]
Roundtrip time: 6.46ms

🪐quickstart>delete from position;
(0 rows) [server: 1.47ms]
Roundtrip time: 7.69ms

🪐quickstart>INSERT INTO position (id, x, y, z) VALUES (1000011, 10.0, 20.0, 30.0);
(0 rows) [inserted: 1, server: 2.82ms]
Roundtrip time: 7.13ms

🪐quickstart>select * from position;
 id      | x  | y  | z
---------+----+----+----
 1000011 | 10 | 20 | 30
(1 row) [server: 1.29ms]
Roundtrip time: 5.49ms

🪐quickstart>update position set x = 1.0;
(0 rows) [updated: 1, server: 1.96ms]
Roundtrip time: 4.50ms

[bfops]

Approving the files that I am a codeowner for:

crates/cli/src/api.rs
crates/cli/src/subcommands/sql.rs

[joshua-spacetime]

Is it possible to add a smoke test as well?

[bfops]

Wait, is this a breaking change? We just added a field, is that right? If it's really a breaking change, we can't release it.

[joshua-spacetime]

That's correct, this just adds a field, and so it's not a breaking change. I believe the label was added by mistake @mamcx?

[mamcx]

That's correct, this just adds a field, and so it's not a breaking change. I believe the label was added by mistake @mamcx?

Yeah, I put the label before adding the serde optionality.