質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.49%

「Pool is closed」のエラーについて

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 2,896

score 47

nodeのmysql2のエラーに関する質問です。
下記のコードをAWSのLambda関数で実行すると、何度かに1度初回のDB接続(check()関数)にて「Pool is closed」のエラーが発生致します。
(問題なく実行できる場合もあります。)

どういう状況でこのエラーが発生するかもわからず困っております。
(当該lambda関数を短い時間(数分内)に連続して実行するとこのエラーが発生しているようにも見えます。)

どの部分を改善すれば「Pool is closed」エラーが発生しなくなるかご教示頂けましたら、幸甚で御座います。

const AWS = require('aws-sdk');
const mysql = require('mysql2');

const pool = mysql.createPool({
    host : 'xxxx',
    user : 'xxxx',
    password : 'xxxx',
    database: 'xxxx',
    port: 'xxx',
});

// 接続
pool.query = util.promisify(pool.query);

exports.lambdaHandler = async (event, context) => {
  const check_res = await check();
  const check_a_res = await check_a();
  const check_b_res = await check_b();
  const check_c_res = await check_c();
  pool.end();
  return check_c_res;
};

function check(){
  const data = ["a", "b"];

  try {
    const results = await pool.query('select id from table where id = ? and name = ?' ,data);
    return results;
  }catch(e){
    console.log("check error");
    pool.end();
    throw e;
  }
}

//function check_a()/check_b()/check_c()もDB接続し、それぞれinsert/update/deleteを含む関数となっております。

エラー

"errorMessage": "Pool is closed.",
"errorType": "Error",
"stackTrace": [
"process.nextTick (/var/task/node_modules/mysql2/lib/pool.js:38:40)",
"_combinedTickCallback (internal/process/next_tick.js:131:7)",
"process._tickDomainCallback (internal/process/next_tick.js:218:9)"
]

環境

AWS lambda関数(node.js : 8.10)

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

0

check_a,b,c各々は最初に作ったpoolを使いまわせばいいと思います。
それらの関数の中で毎回poolに接続するのが良くないと思いました。

やり方としては、ラムダ関数の中でpoolに接続して、check_a,b,cはそのpoolを使いまわして、check_cが終わったらpool.end()するでいいと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/18 17:24

    ご回答ありがとう御座います。
    ということは、私の当初のコードでいいという理解でよろしいでしょうか??

    キャンセル

  • 2019/07/18 17:47 編集

    いえ、ソースコードを見た感じだと、poolに接続しているのがグローバルな位置にあります。
    これだと、pool接続は一回しか行われません。

    一つの案ですが
    exports.lambdaHandler = async (event, context) => {
    const pool = mysql.createPool({
    host : 'xxxx',
    user : 'xxxx',
    password : 'xxxx',
    database: 'xxxx',
    port: 'xxx',
    });


    const check_res = await check(pool);
    const check_a_res = await check_a(pool);
    const check_b_res = await check_b(pool);
    const check_c_res = await check_c(pool);
    pool.end();
    return check_c_res;
    };

    のように書けばいいと思います。
    そのためにはcheck関数は引数にpoolを受け取る必要があります

    キャンセル

  • 2019/07/19 11:31

    詳細なご回答ありがとう御座います!大変よく分かりました!

    キャンセル

  • 2019/07/20 09:58

    解決しましたか?^_^

    キャンセル

0

try句の中をこうしたらどうでしょうか。

const results = await pool.query('select id from table where id = ? and name = ?' ,data);
return results;

↓

const results = await pool.query('select id from table where id = ? and name = ?' ,data);
pool.end();
return results;

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/09 14:01

    ご回答誠に有難う御座います。
    当方の質問に不備があり、大変申し訳御座いません。(質問内容を編集致しました。)
    実際にはcheck()関数の後にもDBに接続する関数がいくつか続いておりまして、はじめにDBとのコネクトし、全ての関数の実行が終わった後にpool.end()している形になっております。
    従いまして、初回のDBアクセス時にはpool.end()しない形に致したく存じます。

    キャンセル

  • 2019/07/09 14:19

    Lambdaを連続して実行するときにエラーがでるんですよね?
    関数実行時のバックエンドは同一ですので、適宜pool.end()するしかないと思いますが...
    または、関数が実行し終わるまで再度関数を実行するのを控える、とか...

    キャンセル

  • 2019/07/09 15:09

    ご返信、誠に有難う御座います。
    連続ではあるのですが、重複した連続ではなく、当該lambda関数が終わった後すぐに実行した際に当該エラーになることがあったという感じです(これも毎回エラーになるわけではないのです)。
    一回一回DBとの接続を切ってというのは、実行時間が掛かるのではないかと思い、避けていたのですが、基本的には一回一回DBとは、接続-切断を繰り返すプログラムの方が良いプログラムなのでしょうか??
    ご教示頂けましたら大変有り難く存じます。

    キャンセル

  • この投稿は削除されました

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る