Skip to content

Change to Sensor from PollingSensor for SQS Sensor #90

Open

Description

@grepory

Right now, the PollingSensor caps at 10 messages per second due to the API limit for maximum messages in a transaction and the minimum polling rate of 1 Hz.

We have modified the SQS sensor locally to be a continuously running Sensor that does its own polling loop instead of a scheduled poll as we have to handle 30+ messages per second. Would this be a welcomed upstream contribution?

Activity

  1. Kami commented on Dec 18, 2019

    @Kami
    Contributor

    Yeah, the contribution would be more than welcome.

    For backward compatibility and similar reasons, it would probably be good to add it as a separate sensor though.

  2. grepory commented on Dec 18, 2019

    @grepory
    Author

    That makes sense to me. Thanks!

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions