Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

[Dart] fix dart3_aot regression (2)#10852

Merged
msmith-techempower merged 4 commits into
TechEmpower:masterfrom
iapicca:dart3_aot_regression_fix_2
Mar 10, 2026
Merged

[Dart] fix dart3_aot regression (2)#10852
msmith-techempower merged 4 commits into
TechEmpower:masterfrom
iapicca:dart3_aot_regression_fix_2

Conversation

@iapicca
Copy link
Copy Markdown
Contributor

@iapicca iapicca commented Mar 8, 2026

Follow-up to #10835

This PR fixes two issues that were silently preventing worker isolates from scaling correctly in AOT mode:

  • Dockerfile misconfiguration: Fixed an issue where the AOT snapshot wasn't being executed properly, preventing the isolate scaling logic from triggering.

  • Dart runtime crash: Fixed a silent crash in the worker processes. The VM passes command-line arguments as a fixed-length list, which caused an UnsupportedError when the child isolates tried to remove the _workerGroup flag. The arguments are now safely copied into a mutable list ([...arguments]) before modification.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 8, 2026

The following frameworks were updated, pinging maintainers:
dart3: @iapicca

@iapicca iapicca changed the title [Dart3] fix dart3_aot regression (2) [Dart] fix dart3_aot regression (2) Mar 8, 2026
@msmith-techempower msmith-techempower merged commit e4e1e4a into TechEmpower:master Mar 10, 2026
3 checks passed
DavideD pushed a commit to DavideD/FrameworkBenchmarks that referenced this pull request Mar 16, 2026
* fix typo in dart3-aot.dockerfile

* fix  unhandled exception

* make workerGroupTag private

* bump debian-distroless version
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants