Skip to content

Add driver API check #129

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

Merged
merged 1 commit into from
Aug 13, 2020
Merged

Add driver API check #129

merged 1 commit into from
Aug 13, 2020

Conversation

LeonidVas
Copy link
Contributor

@LeonidVas LeonidVas commented Aug 3, 2020

In addition to the core queue drivers, customer drivers are exists. In the commit a check for a driver API implementation was added. Now, the consumer will be informed about the missing methods in the driver implementation.

Closes #126

ChangeLog: added notification about missing methods in the driver implementation.
ReleaseNotes: the "tasks_by_state" method was added to the driver API. See example implementation in one of the core drivers.

@LeonidVas LeonidVas force-pushed the lvasiliev/gh-126-fail-gracefully branch from ebf37f9 to 8a3bf9e Compare August 3, 2020 11:48
@LeonidVas LeonidVas requested a review from Totktonada August 3, 2020 11:55
@LeonidVas LeonidVas self-assigned this Aug 3, 2020
@LeonidVas LeonidVas force-pushed the lvasiliev/gh-126-fail-gracefully branch from 8a3bf9e to 8faec74 Compare August 12, 2020 12:13
Copy link
Member

@Totktonada Totktonada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM except one comment below.

In addition to the core queue drivers, customer drivers are exists.
In the commit a check for a driver API implementation was added.
Now, the consumer will be informed about the missing methods in
the driver implementation.

Closes #126
@LeonidVas LeonidVas force-pushed the lvasiliev/gh-126-fail-gracefully branch from 8faec74 to 03701f2 Compare August 13, 2020 13:52
@Totktonada Totktonada merged commit 88a33c9 into master Aug 13, 2020
@Totktonada Totktonada deleted the lvasiliev/gh-126-fail-gracefully branch August 13, 2020 14:32
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.

Fail gracefully at lack of tasks_by_state() driver API method
2 participants