On Wed, May 14, 2014 at 9:52 AM, Zeev Suraski <zeev@zend.com> wrote:
> Well put Nikita!
>
> Guys - we're in a bit of a ridiculous situation where the key low-level
> engine maintainers are saying this patch is unacceptable, yet it may pass
> due to the low level of overall interest and the lack of special rules to
> govern low-level changes like that (where with all due respect, I think the
> main maintainers of the engine should get much stronger power).
>
> The patch the way it is now should be discarded; All the macro changes, as
> well as any data structure change except for IS_LONG should be removed.
>
> Given that the current RFC doesn't thoroughly represent the performance hit
> (in terms of memory footprint, as well as resulting performance hit -
> especially when using phpng), I recommend the following:
>
> * Add the relevant performance feedback from Dmitry to the RFC, as well as
> his concerns as the chief performance guy php.net has
> * Provide an option for people to vote 'yes' for the IS_LONG size part only
>
> If the authors of the RFC object, my request from everyone who has voting
> rights here is to vote 'no' and we can create a separate RFC for the
> IS_LONG
> part only. Forcing this change against the explicit concerns of Dmitry and
> Nikita, who worked their rear ends off to squeeze every bit of performance
> in phpng (along with Xinchen, and now others joining in) - is, well,
> ridiculous IMHO.
>
> Zeev
>
>
Even though that I agree, that our current voting process is suboptimal,
but I would rather see it fixed instead of ignoring the rules when it is
inconvenient/plain wrong.
What I do think is wrong with the current rfc, is that it states that it
needs only 50%+1 of the votes for acceptance.
The patch is big, it will affect every php installation (eg. not just some
platforms), it changes the Zend Engine, and it requires some non-trivial
effort from the extensions to support it, so even though that the direct
impact on the userland isn't that big (code with long strings etc. which
would fatal will now starts working, big numbers will overflow differently,
etc.) I think it still mandates a 66%+1 vote.
Just my 2 cents.
--
Ferenc Kovács
@Tyr43l - http://tyrael.hu