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 for Mariadb or edit MySQL handler to support Mariadb #1619

Open
benmarte opened this issue Feb 16, 2021 · 4 comments
Open

Add support for Mariadb or edit MySQL handler to support Mariadb #1619

benmarte opened this issue Feb 16, 2021 · 4 comments

Comments

@benmarte
Copy link

Is your feature request related to a problem? Please describe.

I'm trying to use graphql-mesh with a mariadb database, I'm using the MySQL handler and following the instructions and when I try to run yarn graphql-mesh serve it gives me this error:

× Unable to start GraphQL-Mesh: ER_PARSE_ERROR: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'keys WHERE Key_name = "PRIMARY"' at line 1
error Command failed with exit code 1.

Describe the solution you'd like
Make a MariaDB handler or edit the MySQL one to handle MariaDB

@benmarte benmarte changed the title Add support for Mariadb or edit MySQL handler for Mariadb Add support for Mariadb or edit MySQL handler to support Mariadb Feb 16, 2021
@ardatan
Copy link
Owner

ardatan commented Feb 18, 2021

@benmarte Thanks for this feature request! Could you try the alpha versions in this comment?
#1627 (comment)

@benmarte
Copy link
Author

@benmarte Thanks for this feature request! Could you try the alpha versions in this comment?
#1627 (comment)

I'd be happy to, just let me know how I would go about getting the alpha change. Thanks.

@benmarte
Copy link
Author

Any update regarding this? Can I get instructions how to get my hands on the alpha version so I can test?

Thanks.

@Witless
Copy link

Witless commented Sep 21, 2021

Hi Ben, the update is in the pull request Ardan attached to his comment.

For what I've seen it's been already merged, I have tried it and I'm getting the same error you get

@theguild-bot theguild-bot mentioned this issue Aug 11, 2022
@theguild-bot theguild-bot mentioned this issue Sep 28, 2023
This was referenced Apr 30, 2024
This was referenced May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants