Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:99624
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain zend.com from 104.47.34.113 cause and error)
To: =?iso-8859-1?Q?Fran=E7ois_Laupretre?= <francois@tekwire.net>, Internals
	<internals@lists.php.net>
Thread-Topic: [PHP-DEV] [RFC][VOTE] Add support for stream-wrapped URLs in
 opcode cache
Thread-Index: AQHS6SfOy7GvQLQlO0+3Pkq3eirKQqIwgkUUgABjVYCAASB7iA==
Date: Fri, 23 Jun 2017 06:48:52 +0000
Message-ID: <BN6PR02MB3234EBB0F2C81EFA7FC640D5BFD80@BN6PR02MB3234.namprd02.prod.outlook.com>
References: <1e99c642-e3b8-3445-82ac-f6cb6258e0b4@php.net>
 <4ef58012-ad0d-4a7b-c558-159a670d2d40@tekwire.net>
 <BN6PR02MB3234EBAF9B38E23061996D47BFDB0@BN6PR02MB3234.namprd02.prod.outlook.com>,<19bcbfa1-4681-578d-2b8e-03e8a923c83f@tekwire.net>
In-Reply-To: <19bcbfa1-4681-578d-2b8e-03e8a923c83f@tekwire.net>
Accept-Language: en-US
Content-Language: en-US
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: multipart/alternative;
	boundary="_000_BN6PR02MB3234EBB0F2C81EFA7FC640D5BFD80BN6PR02MB3234namp_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Jun 2017 06:48:53.0014
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 32210298-c08b-4829-8097-6b12c025a892
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR02MB3233
Subject: Re: [PHP-DEV] [RFC][VOTE] Add support for stream-wrapped URLs in
 opcode cache
From: dmitry@zend.com (Dmitry Stogov)

--_000_BN6PR02MB3234EBB0F2C81EFA7FC640D5BFD80BN6PR02MB3234namp_
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable

This way should work.

Currently it works only for phar:// and file://, because opcache knows them=
, but it's possible to use some stream flag.

I suggest stop voting, re-work proposal, provide implementation, and try on=
ce again.


Thanks. Dmitry.

________________________________
From: Fran=E7ois Laupretre <francois@tekwire.net>
Sent: Thursday, June 22, 2017 4:32:13 PM
To: Dmitry Stogov; Internals
Subject: Re: [PHP-DEV] [RFC][VOTE] Add support for stream-wrapped URLs in o=
pcode cache


Hi Dmitry,

Le 22/06/2017 =E0 09:42, Dmitry Stogov a =E9crit :
The PR is incomplete, so I can't test and even understand the idea complete=
ly.

In my opinion, user defined streams can't be cached, because the wrapper co=
de itself may be changed from request to request.

Right. I hadn't imagined the case where two different applications would us=
e a common wrapper name. So, we should probably restrict the feature to per=
manent (C code) wrappers.

Regards

Fran=E7ois

--_000_BN6PR02MB3234EBB0F2C81EFA7FC640D5BFD80BN6PR02MB3234namp_--