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

[Integration] GreptimeDB database integration #9066

Open
sunng87 opened this issue Apr 12, 2024 · 3 comments · May be fixed by #9071
Open

[Integration] GreptimeDB database integration #9066

sunng87 opened this issue Apr 12, 2024 · 3 comments · May be fixed by #9071
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@sunng87
Copy link

sunng87 commented Apr 12, 2024

GreptimeDB is an open-source time series database. It has built-in support for mysql wire-protocol. I have tested it as a mysql database in mindsdb and it works out of box.

It would be nice to include greptimedb on mindsdb integration. I think there are tasks:

  • Add a new engine called greptimedb although it doesn't require much development and new dependency
  • Add docs on mindsdb's doc site

Please let me know if I can help.

@ZoranPandovski ZoranPandovski added enhancement New feature or request help wanted Extra attention is needed labels Apr 12, 2024
@ZoranPandovski
Copy link
Member

It sounds like it can be supported by just extending the MySQLHandler. For further reference check https://docs.mindsdb.com/contribute/data-handlers

@sunng87
Copy link
Author

sunng87 commented Apr 13, 2024

Oh nice! I will work on this.

@sunng87 sunng87 linked a pull request Apr 13, 2024 that will close this issue
5 tasks
@sunng87
Copy link
Author

sunng87 commented Apr 19, 2024

@ZoranPandovski I've created PR #9071 by extending MySQLHandler and you may want to review it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants