ClickHouse / clickhouse-js

Official JS client for ClickHouse DB
https://clickhouse.com
Apache License 2.0
217 stars 26 forks source link

Add tests for Nested data type #227

Closed auvred closed 8 months ago

auvred commented 8 months ago

Summary

Closes https://github.com/ClickHouse/clickhouse-js/issues/89

It looks like the Nested data type is already supported, but the documentation says it isn't yet supported: https://clickhouse.com/docs/en/integrations/language-clients/javascript#supported-clickhouse-data-types

https://github.com/ClickHouse/clickhouse-docs/blob/d99f31aa0e8aa50085a737ca3402f09931c75b9f/docs/en/integrations/language-clients/js.md?plain=1#L739

This may lead to confusion.

I'm adding tests to prove that it does work, so I can update the status of support in the docs

Checklist

CLAassistant commented 8 months ago

CLA assistant check
All committers have signed the CLA.

slvrtrn commented 8 months ago

Thanks for the contribution!