feat: add interval type support for RowBinaryWithNamesAndTypes

[timvermeulen]

Summary

Adds Interval type support for RowBinaryWithNamesAndTypes.

---

I added conversions to/from SerdeType::I64 because that seems to be how ClickHouse stores interval values, though https://clickhouse.com/docs/sql-reference/data-types/special-data-types/interval says "Time interval as an unsigned integer value.", so let me know if SerdeType::I64 is the appropriate type here.

[slvrtrn]

Thanks for your contribution!

I added conversions to/from SerdeType::I64 because that seems to be how ClickHouse stores interval values

yeah, seems to be a correct assumption:

curl -XPOST "http://localhost:8123?default_format=RowBinaryWithNamesAndTypes" \
  --data-binary "SELECT toIntervalYear(-42) AS y" > out.bin

yields

Since you are changing validation too, could you add more integration tests for that? See https://github.com/ClickHouse/clickhouse-rs/blob/main/tests/it/rbwnat.rs on the current main branch.

I think just a few SELECT queries with min int64, zero, max int64 values for all interval units will be enough.

NB: it is possible to run something like

CREATE TABLE foobar
(
    `i` IntervalYear
)
ENGINE = Memory

which is described as

DESCRIBE TABLE foobar
   ┌─name─┬─type─────────┬─default_type─┬─default_expression─┬─comment─┬─codec_expression─┬─ttl_expression─┐
1. │ i    │ IntervalYear │              │                    │         │                  │                │
   └──────┴──────────────┴──────────────┴────────────────────┴─────────┴──────────────────┴────────────────┘

so we'll need to add integration tests for insert after #244, too

[slvrtrn]

Clippy issues are fixed on main.

[timvermeulen]

Thanks for the pointers! I've added an integration test.