Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112654
Return-Path: <remi@php.net>
Delivered-To: mailing list internals@lists.php.net
Received: (qmail 1342 invoked from network); 30 Dec 2020 07:17:14 -0000
Received: from unknown (HELO php-smtp4.php.net) (45.112.84.5)
  by pb1.pair.com with SMTP; 30 Dec 2020 07:17:14 -0000
Received: from php-smtp4.php.net (localhost [127.0.0.1])
	by php-smtp4.php.net (Postfix) with ESMTP id 31A0C1804C4
	for <internals@lists.php.net>; Tue, 29 Dec 2020 22:51:51 -0800 (PST)
X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on php-smtp4.php.net
X-Spam-Level: 
X-Spam-Status: No, score=0.1 required=5.0 tests=BAYES_20,RCVD_IN_DNSWL_LOW,
	SPF_HELO_NONE,SPF_NEUTRAL autolearn=no autolearn_force=no version=3.4.2
X-Spam-Virus: No
X-Envelope-From: <remi@php.net>
Received: from relay7-d.mail.gandi.net (relay7-d.mail.gandi.net [217.70.183.200])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(No client certificate requested)
	by php-smtp4.php.net (Postfix) with ESMTPS
	for <internals@lists.php.net>; Tue, 29 Dec 2020 22:51:50 -0800 (PST)
X-Originating-IP: 94.238.222.206
Received: from builder.remirepo.net (unknown [94.238.222.206])
	(Authenticated sender: contact@ll-experts.com)
	by relay7-d.mail.gandi.net (Postfix) with ESMTPSA id 8F69820002
	for <internals@lists.php.net>; Wed, 30 Dec 2020 06:51:48 +0000 (UTC)
To: internals@lists.php.net
References: <CAH5C8xVr9zVyyzKAVM=D56-oMVGf9Nsxqa4rq9k1xM+gsH2SPg@mail.gmail.com>
Message-ID: <8c1f2d5a-6888-0787-06ad-095a06dd4e7a@php.net>
Date: Wed, 30 Dec 2020 07:51:47 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.6.0
MIME-Version: 1.0
In-Reply-To: <CAH5C8xVr9zVyyzKAVM=D56-oMVGf9Nsxqa4rq9k1xM+gsH2SPg@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Subject: Re: [PHP-DEV] [RFC] Bundling ext/simdjson into core
From: remi@php.net (Remi Collet)

Le 29/12/2020 à 17:57, Máté Kocsis a écrit :
> Hi Internals,
> 
> I think this will be my last proposal for quite some while :)
> But this time, I'd like to propose bundling the
> https://github.com/crazyxman/simdjson_php extension
> with some major modifications.
> 
> The proposed OO API is included in the description of the
> PR that I've just created: https://github.com/php/php-src/pull/6551
> 
> The main motivation behind this RFC is two-fold:
> - the underlying simdjson library (https://github.com/simdjson/simdjson)
> which is used by ext/simdjson provides huge performance gains
> compared to ext/json (see some benchmark results in the PR)
> - we can support new use-cases, most notably the so called "on-demand"
> parsing: https://github.com/simdjson/simdjson/blob/master/doc/ondemand.md
> (This is not implemented currently)
> 
> Originally, I planned to include the new API in ext/json, but unfortunately,
> simdjson is written is C++, so it would make C++ as a hard dependency,
> which was not the case so far. That's why I opted for creating ext/simdjson.
> 
> Please let me know if you have any feedback.

the library seems young and very active, and so probably not available 
in most distributions.

I would prefer to use the common way

- propose as pecl extension
- wait for maturity (extension AND library)
- propose for merge in php-src

IMHO: too early for this one.

Remi

> 
> Regards:
> Máté
>