feat(query): Async query execution by returning early from long running queries

[pthurlow]

No description provided.

[claude]

Review

Issues

• src/query.rs:136 — The "poll again" hint in poll() is missing the status subcommand. It prints hotdata query <id> but the correct invocation is hotdata query status <id>. The correct hint is already used in execute() at line 82. Following this bad hint would either fail to parse or attempt to run the query-run ID as SQL.

Action Required

Fix line 136: change "Poll again with: hotdata query {}" to "Poll again with: hotdata query status {}".

[claude]

Review

Issues

P1 — src/query.rs:132–137: poll exits 0 when query is still running

The catch-all arm (statuses like "running" or "pending") prints to stderr and returns with exit code 0. This is indistinguishable from a successful result to any caller checking exit codes, making scripted polling impossible without parsing stderr output.

Fix: exit with a non-zero code (e.g. std::process::exit(2)) in the still-running arm so callers can loop on it reliably.

Action Required

Add a non-zero exit in the status => catch-all arm of poll before this can merge.