Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

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

未关闭 适合新手
#24,183 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
73/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
github-actions, php
领域
ci-cd, performance

调研方向

从 test-suite.yml 开始,将 BENCHMARKING 作业的 configure 选项和 apt 软件包与测试作业进行比较。建议的更改是添加 libicu-dev 并启用 intl;验证基准测试作业是否能使用此扩展构建。完成标准是 Symfony Demo 基准测试使用 ext-intl,并按描述注明历史记录的更改。

由索引模型根据 Issue 内容生成。

描述

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.

主要语言
C
星标
40.4k
派生
8.2k
平均合并
1 天 23 小时
30 天内合并 PR
150

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

php/php-src 的其他 Issue

查看 php/php-src 的全部 Issue

相似的 Issue

更多 C Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。