Skip to content

expression: check timezone when encoding timestamp datum (#10303)#10345

Merged
zz-jason merged 2 commits intopingcap:release-2.1from
erjiaqing:merge-10303-v2.1
May 5, 2019
Merged

expression: check timezone when encoding timestamp datum (#10303)#10345
zz-jason merged 2 commits intopingcap:release-2.1from
erjiaqing:merge-10303-v2.1

Conversation

@erjiaqing
Copy link
Copy Markdown
Contributor

Cherry #10303 to release-2.1

@zz-jason
Copy link
Copy Markdown
Member

zz-jason commented May 5, 2019

LGTM

@zz-jason zz-jason added the status/LGT1 Indicates that a PR has LGTM 1. label May 5, 2019
@winkyao winkyao added the priority/release-blocker This issue blocks a release. Please solve it ASAP. label May 5, 2019
Copy link
Copy Markdown
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
Copy link
Copy Markdown
Contributor

alivxxx commented May 5, 2019

/run-all-tests tidb-test=release-2.1 tikv=release-2.1 pd=release-2.1

@alivxxx alivxxx added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels May 5, 2019
@zhouqiang-cl
Copy link
Copy Markdown
Contributor

/run-all-tests tidb-test=release-2.1 tikv=release-2.1 pd=release-2.1

@zhouqiang-cl
Copy link
Copy Markdown
Contributor

/run-integration-compatibility-test tidb-test=release-2.1 tikv=release-2.1 pd=release-2.1

@zz-jason zz-jason merged commit c5811af into pingcap:release-2.1 May 5, 2019
@sre-bot sre-bot added the contribution This PR is from a community contributor. label Dec 18, 2019
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. priority/release-blocker This issue blocks a release. Please solve it ASAP. status/LGT2 Indicates that a PR has LGTM 2. type/bugfix This PR fixes a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants