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

*: Add support "show open tables" #10166

Merged
merged 6 commits into from
Apr 19, 2019
Merged

Conversation

july2993
Copy link
Contributor

What problem does this PR solve?

fix #10070

relate parser pr: pingcap/parser#291

What is changed and how it works?

TiDB has no concept like mysql's "table cache" and "open table"
For simplicity, we just return an empty result with the same structure
as MySQL's SHOW OPEN TABLES

Check List

Tests

  • Unit test
  • Manual test (add detailed scripts or steps below)
    build and run "show open tables" using mysql client

Code changes

Side effects

Related changes

TiDB has no concept like mysql's "table cache" and "open table"
For simplicity, we just return an empty result with the same structure
as MySQL's SHOW OPEN TABLES
@tiancaiamao
Copy link
Contributor

LGTM

@tiancaiamao tiancaiamao added the status/LGT1 Indicates that a PR has LGTM 1. label Apr 18, 2019
@july2993 july2993 requested a review from crazycs520 April 18, 2019 03:35
@shenli
Copy link
Member

shenli commented Apr 18, 2019

LGTM

@shenli
Copy link
Member

shenli commented Apr 18, 2019

/run-all-tests

@codecov
Copy link

codecov bot commented Apr 18, 2019

Codecov Report

Merging #10166 into master will increase coverage by 0.0165%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##            master     #10166        +/-   ##
===============================================
+ Coverage   77.927%   77.9436%   +0.0165%     
===============================================
  Files          407        407                
  Lines        82898      82951        +53     
===============================================
+ Hits         64600      64655        +55     
+ Misses       13509      13505         -4     
- Partials      4789       4791         +2

@shenli
Copy link
Member

shenli commented Apr 18, 2019

/run-all-tests

@july2993
Copy link
Contributor Author

/run-unit-test

@july2993 july2993 added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Apr 18, 2019
@morgo
Copy link
Contributor

morgo commented Apr 18, 2019

LGTM

@july2993
Copy link
Contributor Author

can anyone help approve and merge it?

@morgo
Copy link
Contributor

morgo commented Apr 19, 2019

@july2993 I have approved it.

@morgo morgo merged commit e327f35 into pingcap:master Apr 19, 2019
@july2993 july2993 deleted the hjh/show_open branch April 19, 2019 03:23
zimulala pushed a commit to zimulala/tidb that referenced this pull request May 7, 2019
TiDB has no concept like mysql's "table cache" and "open table"
For simplicity, we just return an empty result with the same structure
as MySQL's SHOW OPEN TABLES
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/LGT2 Indicates that a PR has LGTM 2. type/compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

not support show open table
5 participants