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

Changlog table comment #1138

Merged
merged 3 commits into from
Jul 7, 2022
Merged

Changlog table comment #1138

merged 3 commits into from
Jul 7, 2022

Conversation

timvaillancourt
Copy link
Collaborator

@timvaillancourt timvaillancourt commented Jun 7, 2022

Related issue: #1133

Description

This PR adds the table-level comment gh-ost changlelog to the changelog table, in order to make it easier to find using information_schema.tables

After this PR, the following query should find gh-ost changelog tables:

SELECT table_schema, table_name FROM information_schema.tables
    WHERE table_name LIKE '%_ghc'
    AND table_comment='gh-ost changelog'

In case this PR introduced Go code changes:

  • contributed code is using same conventions as original code
  • script/cibuild returns with no formatting errors, build errors or unit test errors.

@dm-2 dm-2 merged commit 78da0d0 into master Jul 7, 2022
@dm-2 dm-2 deleted the changlog-table-comment branch July 7, 2022 14:47
RainbowDashy pushed a commit to RainbowDashy/gh-ost that referenced this pull request Jul 13, 2022
d-bytebase pushed a commit to bytebase/gh-ost that referenced this pull request Jul 13, 2022
* Merge pull request github#1137 from github/go-mysql-binlog-dead-code

Remove unused code in `go/mysql/binlog.go`

* Merge pull request github#1138 from github/changlog-table-comment

Changlog table comment

* Merge pull request github#1145 from timvaillancourt/golang-ci-staticcheck-linter

`golang-ci`: enable/fix `staticcheck` lint warnings

* Merge pull request github#1146 from github/switch-to-term-package

Switch to golang.org/x/term; go mod vendor

* chore: go mod tidy; go mod vendor

* Merge pull request github#1134 from github/changelog-migrated-state

Add `Migrated` changelog event state

* Merge pull request github#1147 from timvaillancourt/rm-go-os-process-go

Remove unused `go/os/process.go`

* Move `.Kill()` func from inspector to `go/mysql` (github#1148)

* fix: suppress linter

Co-authored-by: dm-2 <45519614+dm-2@users.noreply.github.com>
Co-authored-by: Tim Vaillancourt <tim@timvaillancourt.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants