Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:91039
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.192.174 as permitted sender)
To: Yasuo Ohgaki <yohgaki@ohgaki.net>,
 "internals@lists.php.net" <internals@lists.php.net>
References: <CAGa2bXbg_kebBVg3tSh4uFHZ6nQXRbjUXNQa0JZx9sezQqb5kg@mail.gmail.com>
 <CAGa2bXYr7Xe5fMs_G+bxjCH8PSkJ4Awbtt95Kh9j4f5QbVwevw@mail.gmail.com>
Message-ID: <56AE9C78.4010301@gmail.com>
Date: Sun, 31 Jan 2016 15:44:56 -0800
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:38.0)
 Gecko/20100101 Thunderbird/38.5.1
MIME-Version: 1.0
In-Reply-To: <CAGa2bXYr7Xe5fMs_G+bxjCH8PSkJ4Awbtt95Kh9j4f5QbVwevw@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Re: [RFC Discussion] Precise Session Management
From: smalyshev@gmail.com (Stanislav Malyshev)

Hi!

> Consequences
>  - If script executes session_commit()/session_destroy()/use read only
> option, PHP will not perform GC. i.e. GC function is not called at
> all. Therefore, probability of GC decreases and GC becomes less
> likely.
>  - If script uses multiple session storage, only the last session
> storage performs GC. i.e. Previously used session storage may not call
> GC function at all if this is done for site wide.
>  - User will not see occasional page rendering delay by GC. (This is
> the main reason of this change)

This doesn't look very good. This has potential to break GC for some
applications, especially high-performance ones which are likely to use
read-only mode, and thus for them GC is needed.

I also don't understand why you need this to avoid delay. Right now GC
is performed on shutdown, doesn't it? If it's performed on shutdown, it
would not delay rendering. I'm not sure I understand why you need to
condition this on session being active or not active. Please explain.

-- 
Stas Malyshev
smalyshev@gmail.com