WP_Queryが不当な量のメモリをリークしている


10

以下の関数でWP_Query()を呼び出すたびに、Wordpressは8 MBのメモリをリークします。そして、私はこの関数を頻繁に呼び出すので、物事はかなり速く毛むくじゃらになります... :(結果の$ queryObjectの設定を解除し、定期的にwp_cache_flush()を呼び出そうとしましたが、どちらも効果がないようです。

function get_post_ids_in_taxonomies($taxonomies, &$terms=array()) {
    $post_ids = array();

    $query = gen_query_get_posts_in_taxonomies($taxonomies, $terms);
    // var_dump($query);

    //Perform the query
    $queryObject = new WP_Query($query); //*****THE 8 MEGABYTES IS LEAKED HERE*****

    //For all posts found...
    if($queryObject->have_posts()) {
        while($queryObject->have_posts()) {
            $queryObject->the_post();

            //Get the $post_id by capturing the output of the_ID()
            ob_start();
            the_ID();
            $post_id = (int) ob_get_contents();
            ob_end_clean();

            // echo $post_id."\n";
            $post_ids[] = $post_id;
        }
    }

    unset($queryObject);

    return $post_ids;
}

gen_query_get_posts_in_taxonomies()は:

function gen_query_get_posts_in_taxonomies($taxonomies, &$terms=array()) {
    //General query params
    $query = array(
        'posts_per_page'    => -1,  //Get all posts (no paging)
        'tax_query'             => array('relation' => 'OR'),
    );

    //Add the specific taxonomies and terms onto $query['tax_query']
    foreach($taxonomies as $tax) {
        //Get terms in the taxonomies if we haven't yet
        if(!array_key_exists($tax, $terms)) {
            $terms[$tax] = array();

            $terms_tmp = get_terms($tax);
            foreach($terms_tmp as $tt)
                $terms[$tax][] = $tt->term_taxonomy_id;
        }

        $query['tax_query'][] = array(
            'taxonomy' => $tax,
            'terms' => $terms[$tax],
            'field' => 'term_taxonomy_id',
        );
    }

    return $query;
}

1
DEBUG BARプラグインを試しましたか?
カイザー

WP_Queryあなたのケース(8MBがリークされている場合)では、何件の投稿がフェッチされますか?
ユージーンマヌイロフ

回答:


14

WPハッカーに関する優れた応答:http : //lists.automattic.com/pipermail/wp-hackers/2012-June/043213.html

そのクエリで行っているのは、完全な投稿コンテンツを含むすべての一致する投稿をメモリにロードすることです。ご想像のとおり、これはかなりの数のアイテムです。

'fields' => 'ids'をWP_Queryに渡して、代わりに一致するpost_idのリストを返すだけで、メモリ(および処理時間)を大幅に削減できます。

http://codex.wordpress.org/Class_Reference/WP_Query#Post_Field_Parameters


3

ここで指摘されたメモリの問題を調査しているときにこれに偶然遭遇しました。

この場合、IDを取得するためにバッファリングを使用する代わりにget_the_idを使用でき、IDのみを含めるようにクエリされたフィールドを絞り込むことができます。


回答ありがとうございます、トーマス!私が覚えているように、私は最終的にいくつかの生のSQLを書いてしまいました。ただし、これもおそらく機能します。本当にありがとう!:)
rinogo 2012
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.