Skip to content

Conversation

orian
Copy link
Contributor

@orian orian commented Jun 6, 2025

Summary

Tiny change to ensure the column types are always returned

Checklist

Delete items not relevant to your PR:

  • Unit and integration tests covering the common scenarios were added

@genzgd
Copy link
Collaborator

genzgd commented Jun 6, 2025

Is there a particular reason to add this to the test? It's fine, I just want to understand what prompted the PR.

@orian
Copy link
Contributor Author

orian commented Jun 9, 2025

I've stubled onto this when trying to switch to HTTP interface for querying CH. I was searching for documentation or test proving me wrong, so i've added test to help anyone in the future (or if the behaviour of CH changes)

@genzgd genzgd merged commit ced1ec4 into ClickHouse:main Jun 9, 2025
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants