Skip to content

Allow users of the driver to disable query formatting; support named parameters in QueryStmt, DMLStmt#43

Merged
ohaibbq merged 2 commits intocandidate/rb20240201from
dan/candidate-disable-formatting
Apr 11, 2024
Merged

Allow users of the driver to disable query formatting; support named parameters in QueryStmt, DMLStmt#43
ohaibbq merged 2 commits intocandidate/rb20240201from
dan/candidate-disable-formatting

Conversation

@ohaibbq
Copy link

@ohaibbq ohaibbq commented Apr 10, 2024

As part of goccy/bigquery-emulator#294 we want to be able to disable query formatting for queries in order to fine-tune the query plans generated by SQLite.

This implements an option that can be used to disable formatting as well as supports named parameters.

Copy link

@recidinick recidinick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really cool and really good find!

@ohaibbq ohaibbq changed the title Allow users of the driver to disable query formatting; support named parameters in QueryStmt Allow users of the driver to disable query formatting; support named parameters in QueryStmt, DMLStmt Apr 10, 2024
@ohaibbq ohaibbq merged commit e4a1cd3 into candidate/rb20240201 Apr 11, 2024
ohaibbq added a commit that referenced this pull request Jun 29, 2024
…parameters in `QueryStmt`, `DMLStmt` (#43)

* Allow users of the driver to disable query formatting

* lint
@ohaibbq ohaibbq deleted the dan/candidate-disable-formatting branch July 8, 2024 01:42
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.

3 participants