Запуск параллельного выполнения функций с помощью Promise all - PullRequest
0 голосов
/ 17 марта 2020

У меня есть требование выполнить несколько параллельных функций

Первые функции: Операция получения базы данных ie Найти данные инвентаризации из базы данных

Вторые функции: Обработка результатов в БД и сохранение результата в виде файла

Я использую два обещания все, спина к спине, для выполнения вышеуказанных функций, я не чувствую, что это правильный способ сделать это. Есть ли лучший способ обработки этих вызовов функций. Я получаю результат в соответствии с приведенным ниже кодом, но хочу знать любым другим способом.

Делаем это следующим образом:

let allQuery = {
  sql: "SELECT * from inventory",
};
let inventoryQuery = {
  sql: "SELECT * from inventory where inventory='1'",

};
let nearbyQuery = {
  sql: "SELECT * from inventory where inventory='2",

};
let firstPromises = [dbService.fetch(allQuery),
  dbService.fetch(inventoryQuery),
  dbService.fetch(nearbyQuery)
];
Promise.all(firstPromises)
    .then((values) => {
      let all = values[0];
      let inventory = values[1];
      let nearby = values[2];
      let fileKey1 = folderName + '/' + all.QueryExecutionId + '.csv';
      let fileName1 = all.QueryExecutionId + '.csv';
      let fileKey2 = folderName + '/' + inventory.QueryExecutionId + '.csv';
      let fileName2 = inventory.QueryExecutionId + '.csv';
      let fileKey3 = folderName + '/' + nearby.QueryExecutionId + '.csv';
      let fileName3 = nearby.QueryExecutionId + '.csv';
      let secondPromises = [s3Service.s3StreamDownload(bucketName, fileKey1, fileName1),
        s3Service.s3StreamDownload(bucketName, fileKey2, fileName2),
        s3Service.s3StreamDownload(bucketName, fileKey3, fileName3)
      ];
      Promise.all(secondPromises)
          .then((values) => {
            console.log('Do later operation');

          }).catch((error) => {
        debug(`Error in promises ${error}`);

      });

    }).catch((error) => {
  debug(`Error in promises ${error}`);

});

1 Ответ

2 голосов
/ 17 марта 2020

Я думаю, что будет более читабельно извлечь внутреннюю функцию и затем связать их вместе:

Promise.all(firstPromises)
  .then(transformToSecondPromises)
  .then(Promise.all)
  .then(values => {/* do later operation */})
  .catch(error => { debug(`Error in promises ${error}`) })

function transformToSecondPromises ([all, inventory, nearby]) {
  const fileKey1 = folderName + '/' + all.QueryExecutionId + '.csv';
  const fileName1 = all.QueryExecutionId + '.csv';
  const fileKey2 = folderName + '/' + inventory.QueryExecutionId + '.csv';
  const fileName2 = inventory.QueryExecutionId + '.csv';
  const fileKey3 = folderName + '/' + nearby.QueryExecutionId + '.csv';
  const fileName3 = nearby.QueryExecutionId + '.csv';
  return [
    s3Service.s3StreamDownload(bucketName, fileKey1, fileName1),
    s3Service.s3StreamDownload(bucketName, fileKey2, fileName2),
    s3Service.s3StreamDownload(bucketName, fileKey3, fileName3)
  ];
}
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...