aws-samples / aws-samples/amazon-elasticsearch-lambda-samples
context.succeed would be called multiple times
- Vorherrschende Sprache
- JavaScript
- Sterne
- 386
- Forks
- 170
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
I'm a little confused how the `kinesis_lambda_es.js` example is supposed to work. [Documentation says](http://docs.aws.amazon.com/lambda/latest/dg/nodejs-prog-model-using-old-runtime.html#nodejs-prog-model-context-methods-succeed) that context.succeed, when called, will terminate the lambda.
However, the example [would call context.succeed](https://github.com/awslabs/amazon-elasticsearch-lambda-samples/blob/master/src/kinesis_lambda_es.js#L73) for every HTTP response callback of AWS ElasticSearch. Which is correct? These two sources seem to contradict each other. If the docs.aws.amazon.com site is correct, then wouldn't the context.succeed callback need to occur after ALL ES calls had their callbacks called?
Beitragsleitfaden
Rechercherichtung
Beginne mit src/kinesis_lambda_es.js beim context.succeed-Aufruf ungefähr in Zeile 73 und vergleiche anschließend dessen callback flow mit der verknüpften AWS Lambda context-methods-Dokumentation. Bestätige, ob die completion handling des Beispiels dem dokumentierten Verhalten entspricht, und aktualisiere das Beispiel oder seine Dokumentation, damit der erwartete completion point eindeutig ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- aws, javascript
- Bereich
- backend, cloud
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100