Skip to content

Match @clickhouse/client query binding for TupleParam and typed Map keys #83

Closed

Description

@ShawnChen-Sirius

queryBind currently diverges from @clickhouse/client for compound query params:

  • TupleParam serializes as {'values':[...]} instead of (…), so Tuple(...) and Array(Tuple(...)) params fail.
  • Map keys are always stringified, so new Map([[42, 'v']]) fails for Map(Int32, String).

Expected: serialize TupleParam and Map keys the same way @clickhouse/client-common does. Type mismatches should be rejected by the engine, e.g. a number key for Map(String, ...) should not be silently coerced.

Related fix: #75.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions