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

DROP DATABASE should remove entry from TSDB index #3090

Merged
merged 3 commits into from
Jun 22, 2015

Conversation

otoolep
Copy link
Contributor

@otoolep otoolep commented Jun 22, 2015

Fixes issue #3049

This change removes the database object from the TSBD store's index, when DROP DATABASE is issued. Without this change, old measurement information remained in memory for the database. This change also exposed another panic-producing bug, whereby the statement object should be returned unmodified when RewriteDistinct has nothing to do.

@otoolep
Copy link
Contributor Author

otoolep commented Jun 22, 2015

@pauldix

@otoolep otoolep changed the title Drop database from store DROP DATABASE should remove entry from TSBD index Jun 22, 2015
@otoolep otoolep changed the title DROP DATABASE should remove entry from TSBD index DROP DATABASE should remove entry from TSDB index Jun 22, 2015
@otoolep otoolep added this to the 0.9.1 milestone Jun 22, 2015
@otoolep otoolep self-assigned this Jun 22, 2015
@pauldix
Copy link
Member

pauldix commented Jun 22, 2015

+1

otoolep added a commit that referenced this pull request Jun 22, 2015
DROP DATABASE should remove entry from TSDB index
@otoolep otoolep merged commit 4f3d890 into master Jun 22, 2015
@otoolep otoolep deleted the drop_database_from_store branch June 22, 2015 19:09
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.

2 participants