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

planner: function name in ErrWindowInvalidWindowFuncUse should be lowercase #11520

Merged
merged 3 commits into from
Jul 31, 2019

Conversation

tcmichael
Copy link

What problem does this PR solve?

window function: function name of ErrWindowInvalidWindowFuncUse should be lowercase.
Closes #11508

What is changed and how it works?

warp Function name with strings.ToLower()

Check List

Tests

  • Unit test

@sre-bot sre-bot added the contribution This PR is from a community contributor. label Jul 30, 2019
@sre-bot
Copy link
Contributor

sre-bot commented Jul 30, 2019

Hi contributor, thanks for your PR.

This patch needs to be approved by someone of admins. They should reply with "/ok-to-test" to accept this PR for running test automatically.

@tcmichael tcmichael marked this pull request as ready for review July 30, 2019 05:48
@tcmichael
Copy link
Author

@SunRunAway PTAL

Copy link
Contributor

@SunRunAway SunRunAway left a comment

Choose a reason for hiding this comment

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

LGTM

@SunRunAway SunRunAway requested a review from alivxxx July 30, 2019 05:51
Copy link
Contributor

@alivxxx alivxxx left a comment

Choose a reason for hiding this comment

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

LGTM

@alivxxx alivxxx added the status/can-merge Indicates a PR has been approved by a committer. label Jul 30, 2019
@sre-bot
Copy link
Contributor

sre-bot commented Jul 30, 2019

/run-all-tests

@winoros winoros changed the title window function: function name of ErrWindowInvalidWindowFuncUse should be lowercase.(#11508) planner: function name in ErrWindowInvalidWindowFuncUse should be lowercase Jul 30, 2019
@sre-bot
Copy link
Contributor

sre-bot commented Jul 30, 2019

@tcmichael merge failed.

@SunRunAway
Copy link
Contributor

/run-common-test tidb-test=pr/868
/run-integration-common-test tidb-test=pr/868
/run-unit-test

@SunRunAway
Copy link
Contributor

/run-common-test tidb-test=pr/868
/run-integration-common-test tidb-test=pr/868

1 similar comment
@SunRunAway
Copy link
Contributor

/run-common-test tidb-test=pr/868
/run-integration-common-test tidb-test=pr/868

@codecov
Copy link

codecov bot commented Jul 31, 2019

Codecov Report

Merging #11520 into master will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master     #11520   +/-   ##
===========================================
  Coverage   81.3577%   81.3577%           
===========================================
  Files           425        425           
  Lines         91389      91389           
===========================================
  Hits          74352      74352           
  Misses        11705      11705           
  Partials       5332       5332

@SunRunAway
Copy link
Contributor

/run-all-tests tidb-test=pr/868

@sre-bot
Copy link
Contributor

sre-bot commented Jul 31, 2019

cherry pick to release-3.0 in PR #11535

@sre-bot
Copy link
Contributor

sre-bot commented Jul 31, 2019

cherry pick to release-2.1 failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/expression contribution This PR is from a community contributor. status/can-merge Indicates a PR has been approved by a committer. type/bugfix This PR fixes a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

window function: function name of ErrWindowInvalidWindowFuncUse should be lowercase
4 participants