Skip to content
This repository was archived by the owner on Aug 30, 2019. It is now read-only.

quantizer: allow variable assignments in SQL queries#437

Merged
gbbr merged 3 commits into
masterfrom
gbbr/quantize-assign
Jun 21, 2018
Merged

quantizer: allow variable assignments in SQL queries#437
gbbr merged 3 commits into
masterfrom
gbbr/quantize-assign

Conversation

@gbbr

@gbbr gbbr commented Jun 19, 2018

Copy link
Copy Markdown
Contributor

This change alters the tokenizer to allow := assignments in SELECT
parameters, treating them as regular assignments. Previously this would
cause the parser to return an error and fail to tokenize the query.

Now := will be seen as a valid assignment.

@gbbr
gbbr force-pushed the gbbr/quantize-assign branch 2 times, most recently from a905c13 to 109ae21 Compare June 19, 2018 08:16
@gbbr gbbr added the bug label Jun 19, 2018
@gbbr gbbr modified the milestones: 6.2.1, 6.3.0, next Jun 19, 2018
@gbbr
gbbr force-pushed the gbbr/quantize-assign branch 2 times, most recently from 7fbc1ff to 7d42f46 Compare June 19, 2018 12:00
gbbr added 2 commits June 21, 2018 08:43
This change alters the tokenizer to allow ":=" assignments in SELECT
parameters, treating them as regular assignments. Previously this would
cause the parser to return an error and fail to tokenize the query.
@gbbr
gbbr force-pushed the gbbr/quantize-assign branch from 87cfa17 to 1587ddf Compare June 21, 2018 06:43
@gbbr
gbbr merged commit 8b0cee5 into master Jun 21, 2018
@gbbr
gbbr deleted the gbbr/quantize-assign branch June 21, 2018 08:29
gbbr added a commit that referenced this pull request Jul 12, 2018
* quantizer: allow variable assignments in SQL queries

This change alters the tokenizer to allow ":=" assignments in SELECT
parameters, treating them as regular assignments. Previously this would
cause the parser to return an error and fail to tokenize the query.

* quantizer: no space between ":" and "="
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants