Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:118893
MIME-Version: 1.0
References: <CADDxozmK-qb37v1fE_j92UmO_y9XB2dt3rOmVxY4GdfUXwnoPg@mail.gmail.com>
 <42A5DAC6-253E-478D-808C-36690FA88315@gmail.com>
In-Reply-To: <42A5DAC6-253E-478D-808C-36690FA88315@gmail.com>
Reply-To: Mikhail Galanin <mikhail.galanin@team.bumble.com>
Date: Fri, 28 Oct 2022 10:28:04 +0100
Message-ID: <CADDxoz=z2kQsx2cq5FxQyBM4r+zDOcAoK035RqM811PB13CWfA@mail.gmail.com>
To: Rowan Tommins <rowan.collins@gmail.com>
Cc: internals@lists.php.net
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] Microseconds to error log
From: internals@lists.php.net ("Mikhail Galanin via internals")

Hello everyone,

I tried following Derrek's approach (to introduce a new function).

A couple of considerations:

We used to have some pain when this format changed last time (I can't
remember details, unfortunately) but I don't feel like making an
option to provide backward compatibility is a good idea. Early or
lately, we will need to drop this compatibility and the users face
absolutely the same issue.

I hear people don't want to have this format configurable -- I tend to
agree that it's a bit overcomplication.

I'm not sure if I'm ready to switch to ISO8601 but if we have a
consensus here, I'm open to updating the patch. Basically, I'm open to
everything, please share your opinions.

Pull-request: https://github.com/php/php-src/pull/9844

On Fri, Oct 21, 2022 at 11:46 AM Rowan Tommins <rowan.collins@gmail.com> wr=
ote:
>
> On 21 October 2022 08:47:30 BST, Mikhail Galanin via internals <internals=
@lists.php.net> wrote:
> >Basically, we have quite a high-loaded environment and we really want
> >to see timestamps containing milli-/microseconds in our logs.
>
>
> I'm not knowledgeable enough to comment on the implementation details, bu=
t from a user point of view I agree the feature would be useful. It would d=
efinitely need to be behind an ini setting, though, to avoid existing log p=
arsers failing unexpectedly on the new format.
>
> Regards,
> Hi Mikhail,
>
> --
> Rowan Tommins
> [IMSoP]
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: https://www.php.net/unsub.php
>