aws-samples / aws-samples/amazon-elasticsearch-lambda-samples

context.succeed would be called multiple times

Offen
#7 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.