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

Delete dart3_hybrid test#10780

Merged
msmith-techempower merged 12 commits into
TechEmpower:masterfrom
iapicca:dart_hybrid_socket
Feb 18, 2026
Merged

Delete dart3_hybrid test#10780
msmith-techempower merged 12 commits into
TechEmpower:masterfrom
iapicca:dart_hybrid_socket

Conversation

@iapicca
Copy link
Copy Markdown
Contributor

@iapicca iapicca commented Feb 17, 2026

### Description
This PR provides a comprehensive rewrite of the dart_hybrid implementation (previously introduced in #10634).
The goal is to move beyond theoretical core utilization and achieve higher performances.

### Motivation
The previous implementation underperformed relative to dart_native, despite nominally utilizing all available cores.
This rewrite addresses three specific bottlenecks :
- Eliminated latency overhead from standard TCP proxying.
- Resolved NUMA-related cache thrashing on dual-socket hardware.
- Reduced kernel contention and context-switching under extreme load.

### Changes
- Migrated to Docker Compose from bash script
- Applied explicit cpuset pinning to ensure cache locality.
- Utilized Host Networking to bypass Docker Bridge/NAT overhead.
- Replaced TCP with Unix Domain Sockets (UDS) via tmpfs for zero-latency IPC.
- Separated dart_native, dart_aot, and dart_hybrid to handle increasingly divergent implementations.
- Added documentation for the separate implementation variants to the README.
- Bump dependencies of all dart3 implementations

### Note
Replicas and CPU reservations are hardcoded in docker-compose.yml due to Compose limitations in this environment.


Update

Changes related to the dart_hybrid implementation cannot be merged,
as the project environment does appear to support Docker Compose.
The scope has been revised to remove the dart_hybrid implementation and sanitize the test structure instead.

@joanhey
Copy link
Copy Markdown
Contributor

joanhey commented Feb 17, 2026

Please add yourself to the maintainers in the benchmark_config.json.

So with any change or marked as broken, you will be notified.

Example:

{
"framework": "quarkus",
"maintainers": ["franz1981", "Sanne", "geoand"],
"tests": [
{
"default": {

@iapicca
Copy link
Copy Markdown
Contributor Author

iapicca commented Feb 18, 2026

Changes related to the dart_hybrid implementation cannot be merged,
as the project environment does appear to support Docker Compose.
The scope has been revised to remove the dart_hybrid implementation and sanitize the test structure instead.

@iapicca iapicca changed the title [Dart] update dart_hybrid implementation Delete dart3_hybrid test Feb 18, 2026
@iapicca
Copy link
Copy Markdown
Contributor Author

iapicca commented Feb 18, 2026

Please add yourself to the maintainers in the benchmark_config.json.

So with any change or marked as broken, you will be notified.

Example:

{
"framework": "quarkus",
"maintainers": ["franz1981", "Sanne", "geoand"],
"tests": [
{
"default": {

@joanhey
I've added myself in the maintainers list.
Thank you

@msmith-techempower msmith-techempower merged commit daf5842 into TechEmpower:master Feb 18, 2026
3 checks passed
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.

3 participants