-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add queue version to backing table #78
Comments
What's the version of the package Tom? I haven't done any work in that area of the product for a long time so it is probably a bug. Can you please get a copy of the jobs properties and values at the different stages?
Thanks mate. |
I must have been using a data table written by a version from last year. I've updated RJQ, cleared the table and added new jobs and cannot reproduce the problem. |
Oh, that's good. I had not thought about table versions though. There should be a version written to the table so upgrades can take place. The state document seems like a good choice for that. |
A DB upgrade/migration mechanism would be a killer feature. Perhaps just give warning on mismatched table versions for now? |
So I was testing reanimating and then canceling a terminated job. I reanimated a job, it gets updated correctly and goes to waiting status. Then if I try to cancel the job the update query blows up because it doesn't have the
processCount
field on the row.Error: RethinkDB returned an error at getResultError
first_error: 'No attribute `processCount` in object...
This table might be from an older version of RJQ. Is this a bug or should I try this on a fresher queue table?
The text was updated successfully, but these errors were encountered: