Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added vitess_version as variable #7295

Merged
merged 1 commit into from
Jan 14, 2021

Conversation

Akilan1999
Copy link
Contributor

Signed-off-by: Akilan me@akilan.io

Description

added vitess_version as variable

Related Issue(s)

fixes #3112

Checklist

  • [ x] Tests were added or are not required
  • [ not added] Documentation was added or is not required

Signed-off-by: Akilan <me@akilan.io>
@Akilan1999 Akilan1999 added Type: Enhancement Logical improvement (somewhere between a bug and feature) Component: Query Serving labels Jan 14, 2021
@systay systay merged commit da03789 into vitessio:master Jan 14, 2021
@askdba askdba added this to the v9.0 milestone Jan 19, 2021
}

func TestRewrites(in *testing.T) {
tests := []myTestCase{{
in: "SELECT 42",
expected: "SELECT 42",
// no bindvar needs
}, {
in: "SELECT @@vitess_version",
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this query supposed to work?

Copy link
Contributor

Choose a reason for hiding this comment

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

this works

SELECT @@version_comment;
+----------------------------------------------------------------------------------------------------------------------------------------------+
| @@version_comment                                                                                                                            |
+----------------------------------------------------------------------------------------------------------------------------------------------+
| Version: 11.0.0-SNAPSHOT (Git revision  branch '') built on Fri Apr 30 16:22:55 UTC 2021 by root@buildkitsandbox using go1.15.11 linux/amd64 |
+----------------------------------------------------------------------------------------------------------------------------------------------+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Query Serving Type: Enhancement Logical improvement (somewhere between a bug and feature)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expose variable for systems running on Vitess
4 participants