executemany() does not translate pyformat dict params #807

Summary

Cursor.executemany() with pyformat %(name)s SQL and a list of dicts raises a SQLParseException in crate-python 2.2.0. The pyformat→positional translation introduced in dev is never applied on the bulk code path, raw %(name)s SQL is forwarded to CrateDB's HTTP API, which does not understand pyformat syntax.

Root cause

executemany() does not passes seq_of_parameters as parameters:

# src/crate/client/cursor.py, line 113–121
def executemany(self, sql, seq_of_parameters):
    ...
    self.execute(sql, bulk_parameters=seq_of_parameters)  # ← bulk path

execute() only calls _convert_named_to_positional() when isinstance(parameters, dict) (line 101). The bulk_parameters argument is never inspected for pyformat translation.

As a result, the following JSON body is sent to CrateDB's /_sql endpoint:

{
  "stmt": "INSERT INTO t (name, age) VALUES (%(name)s, %(age)s)",
  "bulk_args": [{"name": "Arthur", "age": 42}]
}

CrateDB's SQL parser rejects the %(...)s syntax:

SQLParseException[Failed to parse source [...]]

Reproduction

import crate.client as crate_client

conn = crate_client.connect("localhost:4200")
cur = conn.cursor()
cur.execute(
    "CREATE TABLE IF NOT EXISTS t (name TEXT PRIMARY KEY, age INT) "
    "WITH (number_of_replicas = 0)"
)

# FAILS
cur.executemany(
    "INSERT INTO t (name, age) VALUES (%(name)s, %(age)s)",
    [
        {"name": "Arthur", "age": 42},
        {"name": "Trillian", "age": 35},
    ],
)

Related Issues:

• Issue Version 2.2.0 has a negative impact on SQLAchemy dialect's parameter handling and results #806
• PR Add named parameter support #795