search

elastic / elastic-serverless-forwarder

Elastic Serverless Forwarder
Other
35 stars 36 forks source link

Upload developer notes: understanding errors and triggers #724

Closed constanca-m closed 4 months ago

constanca-m commented 4 months ago

What does this PR do?

This PR adds some notes to help the maintenance of ESF. Please read the file and the comments for more details.

constanca-m commented 4 months ago

Does it make sense to even set the replay queue as an input the same way we set the other inputs?

The input type of the replay queue should not be set by the user. If the outputs don't matter for the replay queue, then the user should not need to specify them either.

zmoog commented 4 months ago

Does it make sense to even set the replay queue as an input the same way we set the other inputs?

I guess the intent was probably to let users decide when and how to reprocess messages in the replay queue.

The input type of the replay queue should not be set by the user. If the outputs don't matter for the replay queue, then the user should not need to specify them either.

ESF creates the replay queue during the installation, right? To enable reprocessing the messages in the replay queue, users must manually create a trigger to pull the message to replay from the SQS queue to the lambda function, right?

constanca-m commented 4 months ago

ESF creates the replay queue during the installation, right?

Yes.

To enable reprocessing the messages in the replay queue, users must manually create a trigger to pull the message to replay from the SQS queue to the lambda function, right?

Yes.