AWS Lambda不等待DynamoDB的Promise.all()

时间:2018-08-31 15:45:21

标签: javascript node.js amazon-web-services promise aws-lambda

我对NodeJS上的Promises还是陌生的,正在尝试在AWS Lambda中编写一些后端代码来查询包含设备的DynamoDB表。

代码如下:

            var geohash = require('ngeohash');
            var AWS = require('aws-sdk');
            var documentClient = new AWS.DynamoDB.DocumentClient();

            exports.handler = (event, context, callback) => {

                if(event.params.querystring.minLat && event.params.querystring.minLon
                    && event.params.querystring.maxLat && event.params.querystring.maxLon ){

                        var bboxes = geohash.bboxes(event.params.querystring.minLat, event.params.querystring.minLon, event.params.querystring.maxLat, event.params.querystring.maxLon, 6);

                        var params = {
                            "TableName": "Devices",
                            "IndexName": "GeoIndex",
                            "KeyConditionExpression": "GeohashPrefix = :bbox",
                            "ExpressionAttributeValues": {
                                ":bbox": ""
                            },
                        };

                        var devices = [];
                        var promises = [];

                        for (var num in bboxes) {
                            params.ExpressionAttributeValues[":bbox"] = bboxes[num];
                            promises = promises.concat(documentClient.query(params, function(err, data) {
                                if (err) {
                                    console.error("Unable to query. Error:", JSON.stringify(err, null, 2));
                                } else {
                                    console.log("Query succeeded.");
                                    devices = devices.concat(data.Items);
                                    console.log(devices);
                                }
                            }).promise())
                        }
                        Promise.all(promises).then(callback(null, devices));
                    }

                }

这里发生的是Promise.all()实际上并没有等待所有的承诺,而是立即返回。

我在做什么错了?

1 个答案:

答案 0 :(得分:4)

您需要将函数传递给then(),而不是函数的结果。

此:

Promise.all(promises).then(callback(null, devices));

应该更像这样:

Promise.all(promises).then(() => callback(null, devices));

在第一种情况下,在定义了诺言后立即调用callback