Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:129441
Precedence: list
MIME-Version: 1.0
References: <CAMW7n8Dogjj2JFJBNArk-BfvOHRqHKMAFReXJAhDm2UPV1TF7A@mail.gmail.com>
 <980296152.1071.1763813573083@email.ionos.de> <CAMW7n8DXA501uGOY3952y8YhAJDVRrPYEiuw+Wzc5=oEciLVZQ@mail.gmail.com>
 <92865666.4510.1763818506332@email.ionos.de> <CAMW7n8AycfX4NMfjJ+GzFsPqRaSae-t7Cm3H5z8apRzXUgaVZw@mail.gmail.com>
 <329450798.8037.1763822426377@email.ionos.de> <CAMW7n8DTt2S_RfcyshERixed=qqyFUPEvWp_MpgpW_CvH3urXQ@mail.gmail.com>
 <CAMW7n8CUQxqh4_rTWQBcj+t17i5oW7x-V96FSkwZL85Z2pzrRw@mail.gmail.com>
 <CAB9bCb34cmu9=an9K4rubPSVEDwyWH-rMAVtTTYa3U-LA+GJ+A@mail.gmail.com>
 <CAMW7n8DM4MCP25gSLBbd3+Awzs2+p6Qs_jPgx1mk=RjSVe+iUw@mail.gmail.com>
 <9287c46c-bc63-4dd0-9792-0f9421959589@rwec.co.uk> <CAMW7n8AhbBawCBtfNJZ6XzFSvwhtZotfgBmb1O4CDZ+J2jxt2w@mail.gmail.com>
 <C766271D-EE7A-4727-8A7C-B117429768E5@rwec.co.uk> <CAMW7n8Bk+AhbXjg8QADFo=2k9qCE54DD68Hxkf2k_9J=EFboUQ@mail.gmail.com>
 <65869feb-d518-4de3-8c10-115e3ba7dce7@rwec.co.uk> <CAMW7n8CA6gfdWzhVCY3j4w_=S-rZW5sUvkSL_e4+hc_Ka+r1rA@mail.gmail.com>
 <55149f3e-7ec7-4479-bd6d-2e7fe1b8edef@rwec.co.uk>
In-Reply-To: <55149f3e-7ec7-4479-bd6d-2e7fe1b8edef@rwec.co.uk>
Date: Mon, 24 Nov 2025 18:57:46 +0200
Message-ID: <CAMW7n8Cfs9+wEpGVp5C2VL3ey8vKUNS2NyJAq+w7-DJXg172Dw@mail.gmail.com>
Subject: Re: [PHP-DEV] [VOTE] True Async RFC 1.6
To: "Rowan Tommins [IMSoP]" <imsop.php@rwec.co.uk>
Cc: internals@lists.php.net
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
From: edmond.ht@gmail.com (Edmond Dantes)

> An error will occur for whom? SDK Susie when unit testing her library?
If SDK Susie used a new coroutine in which she calls the
`LoggerInterface::flush()` method, then with some probability the user
may notice that log entries are sometimes duplicated because the
`LoggerInterface::logs` property is not being cleared.
You could say that SDK Susie is responsible for this, because she ran
code that was never intended to be executed in a concurrent
environment.

> If it is an error for SDK Susie, what is her solution to allow users to p=
ass in a logger to her library?
To avoid the error, SDK Susie must be sure that the
**LoggerInterface** implementation supports concurrent execution.

If SDK Susie knows that LoggerInterface does not support asynchronous
code, then she should not attempt to call its methods in a separate
coroutine and should use LoggerInterface in the usual synchronous way.
The shared-nothing memory model doesn=E2=80=99t even give SDK Susie a chanc=
e
to make this mistake, because it simply forbids the existence of an
object with two references.

> If it is an error for Legacy Les, how does he fix it?

In this case, we should assume that the responsibility for correctness
lies with the one who provides the new functionality. Therefore,
Legacy Les should not have to do anything.
However, if Legacy Les wants to use the library, he needs to do the followi=
ng:

```php
function flush(): void {
     $data =3D $this->logs;
     $this->logs =3D []; // It is guaranteed to execute without interruptio=
ns.
     file_put_contents(implode("\n", $data), FILE_APPEND);
}
```

I want to point out right away that the code above is not suitable for
multithreaded execution.