Skip to content
This repository has been archived by the owner on Nov 28, 2023. It is now read-only.

StreamSubscription.onDataSiftLogMessage Is Never Called #82

Open
michaelahlers opened this issue Jan 16, 2015 · 5 comments
Open

StreamSubscription.onDataSiftLogMessage Is Never Called #82

michaelahlers opened this issue Jan 16, 2015 · 5 comments
Assignees

Comments

@michaelahlers
Copy link

A trivial implementation of this method never produces any output, whereas StreamSubscription.onMessage is regularly invoked. This method is important to my project for confirmation of stream subscriptions.

@michaelahlers
Copy link
Author

A breakpoint (with a condition that msg.data() does not contain string “interaction”) on line 205 of DataSiftConnection never gets hit. I couldn't determine from the documentation one way or another, but does the service issue status messages at all for multi-streams?

@zcourts
Copy link
Contributor

zcourts commented Jan 21, 2015

See http://dev.datasift.com/docs/resources/errors and http://dev.datasift.com/docs/deliver/streaming/multiple-streams the only time you get those messages is when something goes wrong or needs your attention. That method shouldn't be called regularly.

@zcourts zcourts closed this as completed Jan 21, 2015
@michaelahlers
Copy link
Author

@zcourts, one feature our users need is a strong indication that a stream to DataSift is open and healthy—even if their query produces no results. It's also useful to engineers for debugging. With subscribing to a single stream over a single direct WebSocket connection, it was easy to accomplish this using the ping frames (per the Websockets streaming documentation). Just to confirm what you've explained: there is nothing analogous for multi-stream connections?

@dugjason
Copy link
Contributor

@michaelahlers we don't have any available features like the one you've described at present, however it does sound incredibly useful. I'll raise it with our product team as a feature request; we'll have to implement API support for it before we can support it in our client libraries

@dugjason dugjason reopened this Jan 23, 2015
@michaelahlers
Copy link
Author

@dugjason, thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants