CLI - Fix sql --interactive case where returned rows are 0

[bfops]

Description of Changes

See #2358 for more context on this issue and why this was needed.

I think we recently removed the functionality where we return row counts for update and delete statements. This means that the CLI can no longer say things like "N rows updated".

API and ABI breaking changes

No.

Expected complexity level and risk

1

Testing

• No longer crashes when running update or delete commands in interactive mode:

$ "$WORK"/SpacetimeDBPrivate/public/target/debug/spacetimedb-cli sql -s local test-project --interactive
WARNING: This command is UNSTABLE and subject to breaking changes.

┌──────────────────────────────────────────────────────────┐
│ .exit: Exit the REPL                                     │
│ .clear: Clear the Screen                                 │
│                                                          │
│ Give us feedback in our Discord server:                  │
│    https://discord.gg/w2DVqNZXdN                         │
└──────────────────────────────────────────────────────────┘
🪐test-project>update person set name='foo'

Time: 1.28ms
🪐test-project>delete from person

Time: 1.34ms
🪐test-project>

[jdetter]

Suggested change

	if row_count > 0 {
	let row_count = print_row_count(row_count);
	let row_count = print_row_count(row_count);
	if row_count > 0 {

I think you should move this up one line outside of the if because then you would still get the output:

0 rows

Instead of getting no output.

[bfops]

The function is poorly named. It doesn't actually print anything. It just formats it as a string.

[bfops]

even if it were the case, I think that would be misleading. I believe we've just stopped returning the row count from the API calls, but in fact we are updating/deleting rows.