WIP : db_extensible plugin#2286
Closed
menardorama wants to merge 13 commits intoinfluxdata:masterfrom
menardorama:master
Closed
WIP : db_extensible plugin#2286menardorama wants to merge 13 commits intoinfluxdata:masterfrom menardorama:master
menardorama wants to merge 13 commits intoinfluxdata:masterfrom
menardorama:master
Conversation
…only mysql is added
Now useless because you can specify multiple times the same plugin so configuration is simpler (one plugin per database version).
Define a sample test
add blank identifier for db drivers
|
@menardorama if you need beta tester, dont hesitate for MySQL part ;) |
bakl
approved these changes
Feb 1, 2017
|
@bakl what are the next step for merge those commit into the master ? |
|
@sparrc any idea when this feature will be integrated ? @bakl approuved changes |
|
@menardorama any plans for SQLite support? |
|
@menardorama can you check if this pull can cover your needs ?#2785 |
Contributor
|
Closing this in favor of #2785 . |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi,
I've forked the postgresql_extensible plugin in order to add more databases.
For now it's just a WIP, I wanted to have feedbacks on it.
I've just added mysql in the loop for now, but had to comment the sanitize function (need to work with the ParseDSN) for now.
Feedbacks are welcome.