Skip to content

Conversation

lysu
Copy link
Contributor

@lysu lysu commented Aug 23, 2018

What is changed and how it works?

cherry-pick #7355

Check List

Tests

  • Unit test
  • Integration test

Code changes

  • Impl change

Side effects

  • no

Related changes

  • Need to be included in the release note

This change is Reviewable

@lysu
Copy link
Contributor Author

lysu commented Aug 23, 2018

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

Copy link
Contributor

@crazycs520 crazycs520 left a comment

Choose a reason for hiding this comment

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

LGTM

@zz-jason
Copy link
Member

LGTM

@lysu lysu added status/LGT2 Indicates that a PR has LGTM 2. status/all tests passed labels Aug 23, 2018
@ngaut ngaut merged commit d19300f into pingcap:release-2.0 Aug 23, 2018
@lysu lysu deleted the pick-current_time branch September 27, 2018 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/expression status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants