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

Attempt to fix mysql schema #304

Merged
merged 1 commit into from
Jun 21, 2020
Merged

Attempt to fix mysql schema #304

merged 1 commit into from
Jun 21, 2020

Conversation

sagikazarmark
Copy link
Contributor

No description provided.

@sagikazarmark sagikazarmark mentioned this pull request Jun 21, 2020
Copy link
Collaborator

@markphelps markphelps left a comment

Choose a reason for hiding this comment

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

💯 thank you!

@markphelps markphelps merged commit 9f76432 into flipt-io:mysql2 Jun 21, 2020
@sagikazarmark sagikazarmark deleted the mysql2-fix branch June 21, 2020 18:26
@sagikazarmark
Copy link
Contributor Author

It looks like this is required for mysql 5.x support, but the original syntax is compatible with mysql 8

markphelps pushed a commit that referenced this pull request Jun 29, 2020
* WIP

* Redo

* Postgres

* sqlite

* Fix tests

* Rm unneeded stuff

* More mysql

* Migration file

* WIP

* Single store; TODO: caching

* Use dburl lib

* Fix mysql url

* Rm healthcheck command

* Reparse url

* Root

* Switch back pg

* need to fix migrations

* Update migrator

* Getting there

* Fix mysql timestamps

* Add back assert errors

* missed one

* Add back cache support

* Almost there

* fix sorting for distributions

* Wat

* Mod tidy

* current_timestamp

* Attempt to fix mysql schema (#304)

* Use double quotes in queries

* Try and fix pq err code

* Use pq err code name

* Forgot one

* Back to backticks

* Come on Postgres

* pointers

* OMG

* sleeps

* Unembed store in server; fix tests; mod tidy

* Moving things

* Add Example; fix migrator

* Bump go-version

* Add migrate test

* Fix mysql ports; bump changelog

* Update README

* Comment out failing tests for now

Co-authored-by: Márk Sági-Kazár <sagikazarmark@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants