Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:54684
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain managedit.ie designates 209.85.210.172 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAKxcST-KbNk=CyBOg_uLb85FVf0rPS8A7VmQVXnHqdghsXWDLg@mail.gmail.com>
References: <CAKxcST-n_aC1NJ8pLg5natwRRcxEHV_t-fiEhYX4mc=G7WPFaA@mail.gmail.com>
	<CA+p1TcZ07HVpQn7YZngsREYtV-gbtQAPg8Ou4HyVuxE36Pnj7A@mail.gmail.com>
	<CAAP_MjX-FBxyiVd8fccUuA-B5XJwZ_ubr9+wX8fNDiBtRYghFg@mail.gmail.com>
	<CAKxcST-KbNk=CyBOg_uLb85FVf0rPS8A7VmQVXnHqdghsXWDLg@mail.gmail.com>
Date: Thu, 18 Aug 2011 13:25:25 +0100
Message-ID: <CAGSj+QuoLgWHYN0fvXXb5AQn=Xy=PAeSXT9=A3--mO--ZqnOVw@mail.gmail.com>
To: Paul Dragoonis <dragoonis@gmail.com>
Cc: Arpad Ray <arraypad@gmail.com>, Roger Llopart Pla <lumbendil@gmail.com>, 
	PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=bcaec52996bb14596604aac6b5ba
Subject: Re: [PHP-DEV] Activation of IGBinary serialization extension in 5.4
 by default
From: kiall@managedit.ie (Kiall Mac Innes)

--bcaec52996bb14596604aac6b5ba
Content-Type: text/plain; charset=ISO-8859-1

Some others systems use something similar to syntax checking (ldap password
hashes).

Baisically, prefix a code to the serialized string to indicate what method
was used to serialize.

Eg no prefix, or $O$ for the old style.

Or $I$ for the igbinary syntax.

This has the advantage of allowing additional formats to be added over time
without breaking BC.

Kiall
 On Aug 18, 2011 12:49 p.m., "Paul Dragoonis" <dragoonis@gmail.com> wrote:
> On Thu, Aug 18, 2011 at 12:38 PM, Roger Llopart Pla <lumbendil@gmail.com
>wrote:
>
>> I agree with bundling it in the core aswell, but having it as default, as
>> stated, would be a huge BC break.
>>
>> 2011/8/18 Arpad Ray <arraypad@gmail.com>
>>
>>> Hi,
>>>
>>> On Thu, Aug 18, 2011 at 12:12 PM, Paul Dragoonis <dragoonis@gmail.com>
>>> wrote:
>>> > Hey guys,
>>> >
>>> > After a brief discussion with Pierre I'm taking this discussion to
here.
>>> > We have the igbinary pecl extension available [1], however not many of
>>> our
>>> > userbase are familiar that it even exists.
>>> > Here [2] is the performance boost we get from using igbinary with
>>> > serialize() and unserialize().
>>> >
>>> > Here we get a great perf boost, and what's more excellent as we don't
>>> have
>>> > to break BC to get there!
>>>
>>> It would break BC for anyone with serialized strings stored in the
>>> current format.
>>>
>>> Maybe bundle it in core and provide an ini setting like
>>> session.serialize_handler, but I don't think making it the default in
>>> 5.4 is an option.
>>>
>>
> Sorry guys, I didn't realise there was an actual BC break in the
serialized
> structure.
> I agree with bundling and providing a .ini setting to activate, but off by
> default.
>
>
>>
>>> Regards,
>>>
>>> Arpad
>>>
>>> >
>>> > This seems like a no-brainer and a good investment in the speed
increase
>>> of
>>> > PHP by default for the future users of the platform.
>>> >
>>> > [1] http://pecl.php.net/package/igbinary
>>> > [2] http://www.php.net/~pierre/vcqa/apcigninary_perf.png
>>> >
>>> > Comments please.
>>> >
>>> > Regards,
>>> > Paul Dragoonis.
>>> >
>>>
>>> --
>>> PHP Internals - PHP Runtime Development Mailing List
>>> To unsubscribe, visit: http://www.php.net/unsub.php
>>>
>>>
>>

--bcaec52996bb14596604aac6b5ba--