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

Changed configurations to use DatabaseConfigurator stand-in plug-in #4

Merged
merged 1 commit into from Jan 26, 2017

Conversation

srp3rd
Copy link
Contributor

@srp3rd srp3rd commented Jan 25, 2017

The DatabaseConfigurator is a base class that is a placeholder for
a "real" implementation of a database plugin. Since at this time we
don't need to have any interactions with the database (because a
replacement for provenance hasn't been implemented), this information
is unused. It could be deleted without causing errors in orchestration,
but is being kept here because it will be used in the future.

The DatabaseConfigurator is a base class that is a placeholder for
a "real" implementation of a database plugin.  Since at this time we
don't need to have any interactions with the database (because a
replacement for provenance hasn't been implemented), this information
is unused.  It could be deleted without causing errors in orchestration,
but is being kept here because it will be used in the future.
@srp3rd srp3rd merged commit 1d2da23 into master Jan 26, 2017
@ktlim ktlim deleted the tickets/DM-8947 branch August 25, 2018 05:06
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