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

Extend Feed Provider API to include update & status operations? #1925

Closed
jthomas opened this Issue Mar 1, 2017 · 3 comments

Comments

Projects
None yet
4 participants
@jthomas
Member

jthomas commented Mar 1, 2017

Once you have registered a trigger with a feed provider, there's no way to retrieve any details about the current trigger feed state.

For example, with the alarm package, it would be useful to be query the trigger feed to recover the cron value. It would also be nice to have a way to modify feed parameters without having to delete & re-create as this might introduce missed events between the delete finishing and create registered.

Feed Actions currently supported two operations: CREATE and DELETE. Could we add RETRIEVE and UPDATE?

I could see that you could handle UPDATE by issuing another create, without the delete, and expecting providers to handle create requests for existing triggers.

@jberstler

This comment has been minimized.

Show comment
Hide comment
@jberstler

jberstler Apr 27, 2017

One proposal is that each feed package could include a "health" or "status" action which, when invoked, returns the kind of information you are asking for.

jberstler commented Apr 27, 2017

One proposal is that each feed package could include a "health" or "status" action which, when invoked, returns the kind of information you are asking for.

@rabbah

This comment has been minimized.

Show comment
Hide comment
@rabbah
Member

rabbah commented Nov 25, 2017

@csantanapr done?

@csantanapr

This comment has been minimized.

Show comment
Hide comment
@csantanapr

csantanapr Nov 26, 2017

Contributor

Yes done

Contributor

csantanapr commented Nov 26, 2017

Yes done

@csantanapr csantanapr closed this Nov 26, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment