Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Benchmark CI builds Symfony Demo without ext-intl, so 18% of the request is a polyfill

Ouverte Adaptée aux débutants
#24,183 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
73/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
github-actions, php
Domaine
ci-cd, performance

Piste de recherche

Commencez par test-suite.yml et comparez les options de configure et les paquets apt du job BENCHMARKING avec ceux des jobs de test. La modification proposée consiste à ajouter libicu-dev et à activer intl ; vérifiez que le job de benchmark est compilé avec cette extension. Le travail est terminé lorsque le benchmark Symfony Demo utilise ext-intl et que la modification de l’historique est notée comme décrit.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Feature Status: Needs Triage
Description

The BENCHMARKING job in test-suite.yml configures without --enable-intl (no libicu-dev in its apt list either), unlike the test jobs. Symfony Demo then runs symfony/polyfill-intl-* for date formatting, ICU messages and symfony/string; the grapheme polyfill does preg_match_all('/\X/u', …) once per grapheme, 17.8K matches per request.

Callgrind on the job's exact extension set (reproduces the CI counts within 0.5%) vs the same build with ext-intl:

Symfony Demo instructions per request PCRE share
as in CI 39,427,533 19.5%
with ext-intl 31,533,221 (−20%) 1.6%

preg_match_all() alone is 18% of the official number. A Symfony deployment doesn't run that path — the polyfill is the fallback — so a fifth of the benchmark moves with PCRE's global-match loop and \X and says nothing about the engine. bench.php and WordPress are unaffected.

I'd add libicu-dev and --enable-intl to the job. The cost is a −20% step in the Symfony history at the switch, worth a note in the results. I can send the workflow change if that's acceptable.

Langage dominant
C
Étoiles
40.4k
Forks
8.2k
Merge moyen
1 j 23 h
PR mergées (30 j)
150

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de php/php-src

Toutes les issues de php/php-src

Issues similaires

Plus d'issues C

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.