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

fix!: columns table in information_schema misses some columns #3639

Merged
merged 13 commits into from
Apr 8, 2024

Conversation

killme2008
Copy link
Contributor

@killme2008 killme2008 commented Apr 4, 2024

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

Fix #3606

What's changed and what's your intention?

Checklist

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.
  • This PR does not require documentation updates.

@github-actions github-actions bot added the docs-required This change requires docs update. label Apr 4, 2024
@killme2008 killme2008 marked this pull request as ready for review April 4, 2024 03:27
@killme2008 killme2008 changed the title fix: columns table in information_schema misses some columns fix: !columns table in information_schema misses some columns Apr 4, 2024
@killme2008 killme2008 changed the title fix: !columns table in information_schema misses some columns fix!: columns table in information_schema misses some columns Apr 4, 2024
@github-actions github-actions bot added the breaking-change This pull request contains breaking changes. label Apr 4, 2024
Copy link

codecov bot commented Apr 4, 2024

Codecov Report

Attention: Patch coverage is 87.13450% with 22 lines in your changes are missing coverage. Please review.

Project coverage is 84.84%. Comparing base (097a037) to head (e3bce34).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3639      +/-   ##
==========================================
- Coverage   85.14%   84.84%   -0.30%     
==========================================
  Files         943      943              
  Lines      157092   157225     +133     
==========================================
- Hits       133754   133401     -353     
- Misses      23338    23824     +486     

Copy link
Member

@sunng87 sunng87 left a comment

Choose a reason for hiding this comment

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

Do we need full compatibility with MySQL? SRS_ID is still missing although it's all NULL for us.

@killme2008
Copy link
Contributor Author

Do we need full compatibility with MySQL? SRS_ID is still missing although it's all NULL for us.

Yes, forgot this column.

@killme2008 killme2008 requested a review from sunng87 April 7, 2024 09:00
@killme2008
Copy link
Contributor Author

@fengjiachun @waynexia Please take a look

Copy link
Collaborator

@fengjiachun fengjiachun left a comment

Choose a reason for hiding this comment

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

LGTM

src/datatypes/src/data_type.rs Outdated Show resolved Hide resolved
src/query/src/sql.rs Outdated Show resolved Hide resolved
killme2008 and others added 8 commits April 8, 2024 08:54
Co-authored-by: JeremyHi <[email protected]>
Bumps [h2](https://github.com/hyperium/h2) from 0.3.24 to 0.3.26.
- [Release notes](https://github.com/hyperium/h2/releases)
- [Changelog](https://github.com/hyperium/h2/blob/v0.3.26/CHANGELOG.md)
- [Commits](hyperium/h2@v0.3.24...v0.3.26)

---
updated-dependencies:
- dependency-name: h2
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [whoami](https://github.com/ardaku/whoami) from 1.4.1 to 1.5.1.
- [Changelog](https://github.com/ardaku/whoami/blob/v1/CHANGELOG.md)
- [Commits](ardaku/whoami@v1.4.1...v1.5.1)

---
updated-dependencies:
- dependency-name: whoami
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: adding victoria metrics remote write

* test: add e2e tests for prom and vm remote writes
* fix: construct correct pk list with pre-existing pk

Signed-off-by: Ruihang Xia <[email protected]>

* update UT

Signed-off-by: Ruihang Xia <[email protected]>

---------

Signed-off-by: Ruihang Xia <[email protected]>
@killme2008
Copy link
Contributor Author

@sunng87 PTAL

@sunng87 sunng87 added this pull request to the merge queue Apr 8, 2024
Merged via the queue into GreptimeTeam:main with commit 87e0189 Apr 8, 2024
21 checks passed
@killme2008 killme2008 deleted the fix/column-tables branch April 8, 2024 04:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change This pull request contains breaking changes. docs-required This change requires docs update.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The columns table in information_schema is not compatible with MySQL
5 participants