Skip to content

DX-113627: Make RunSqlQuery JSON-safe output#83

Merged
aniket-s-kulkarni merged 1 commit into
dremio:mainfrom
maxlepikhin:fix/run-sql-json-safe
Jan 24, 2026
Merged

DX-113627: Make RunSqlQuery JSON-safe output#83
aniket-s-kulkarni merged 1 commit into
dremio:mainfrom
maxlepikhin:fix/run-sql-json-safe

Conversation

@maxlepikhin
Copy link
Copy Markdown
Contributor

Codex implemented it and tested based on the feedback from Jorge:

The SQL is "SELECT * FROM reporting.telemetry.stg_telemetry_metrics WHERE LOWER(org_name) = 'mobilityware'" and it was originally created by ChatGPT but fails both with Claude and ChatGPT. The error is basically "Error executing tool RunSqlQuery: Error serializing to JSON: TypeError: 'float' object cannot be interpreted as an integer" for both LLMs. It runs fine with the Dremio AI Agent. I wrote an equivalent SQL and it worked fine on all agents (ChatGPT, Claude and the Dremio AI Agent).

Change in tests/conftest.py was required to run tests, otherwise they didn't run with python 3.12.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 23, 2026

CLA assistant check
All committers have signed the CLA.

@maxlepikhin maxlepikhin changed the title Make RunSqlQuery JSON-safe output DX-113627: Make RunSqlQuery JSON-safe output Jan 23, 2026
@aniket-s-kulkarni aniket-s-kulkarni merged commit c0cbb0f into dremio:main Jan 24, 2026
3 checks passed
@maxlepikhin maxlepikhin deleted the fix/run-sql-json-safe branch January 24, 2026 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants