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

context.succeed would be called multiple times

オープン
#7 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
386
フォーク
170
PR マージ指標
30日以内にマージされた PR はありません

説明

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?

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

src/kinesis_lambda_es.js の73行目付近にある context.succeed の呼び出しから始め、続いてその callback のフローをリンク先の AWS Lambda context-methods ドキュメントと比較してください。例の完了処理がドキュメントに記載された動作と一致しているか確認し、期待される完了時点が曖昧にならないように、例またはそのドキュメントを更新してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
aws, javascript
領域
backend, cloud
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。