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

[0.10] can't run CQs with the same name but scoped to different databases #5814

Closed
beckettsean opened this issue Feb 23, 2016 · 2 comments

Comments

Projects
None yet
2 participants
@beckettsean
Copy link
Contributor

commented Feb 23, 2016

From mailing list: https://groups.google.com/d/msgid/influxdb/7b74ce6c-d841-43f5-aeca-39fb860cdeec%40googlegroups.com

If multiple CQs exist with the same name but on different databases, only one CQ is actually executed.

@jsternberg

This comment has been minimized.

Copy link
Contributor

commented Feb 24, 2016

It looks like the map storing the last run doesn't include the database name, so the CQ name isn't scoped there.

Preparing a fix for this. Do we need to backport this and do a bugfix release for 0.10 or can this just go directly into master and wait for 0.11?

@jsternberg jsternberg added this to the 0.11.0 milestone Feb 24, 2016

jsternberg added a commit that referenced this issue Feb 24, 2016

Fix running multiple CQs with the same name
Previously, CQs with the same name would be stored in the last run map
the same way. This caused only one of the CQs to run because after the
first one ran it would update the last run time for all CQs with the
same name.

Add the database name to the CQ ID in the last run map to differentiate
between CQs in different databases.

Fixes #5814.
@beckettsean

This comment has been minimized.

Copy link
Contributor Author

commented Feb 24, 2016

@jsternberg this is a pretty minor issue with a simple workaround, so I see no need to backport to 0.10. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.