Skip to content

harrallo#265

Open
harrallo wants to merge 6 commits into
tempestphp:mainfrom
harrallo:main
Open

harrallo#265
harrallo wants to merge 6 commits into
tempestphp:mainfrom
harrallo:main

Conversation

@harrallo

@harrallo harrallo commented Mar 11, 2026

Copy link
Copy Markdown

No description provided.

@harrallo harrallo closed this Mar 14, 2026
@harrallo harrallo reopened this Mar 14, 2026
@harrallo

Copy link
Copy Markdown
Author

/bench

@brendt

brendt commented Mar 14, 2026

Copy link
Copy Markdown
Member

Benchmarking failed: Parsed result did not match expected result

@harrallo

Copy link
Copy Markdown
Author

last try

@harrallo

Copy link
Copy Markdown
Author

/bench

@brendt

brendt commented Mar 14, 2026

Copy link
Copy Markdown
Member

Benchmarking complete! Best execution time: 11.331s

Full results:

{
  "results": [
    {
      "command": "cd /Users/brentroose/Dev/100-million-row-challenge/app/Commands/../../.benchmark/pr-265 && php -dmax_execution_time=300 tempest data:parse --input-path=\"/Users/brentroose/Dev/100-million-row-challenge/app/Commands/../../data/real-data.csv\" --output-path=\"/Users/brentroose/Dev/100-million-row-challenge/app/Commands/../../data/real-data-actual.json\"",
      "mean": 11.33149160866,
      "stddev": null,
      "median": 11.33149160866,
      "user": 10.79607298,
      "system": 0.50967532,
      "min": 11.33149160866,
      "max": 11.33149160866,
      "times": [
        11.33149160866
      ],
      "memory_usage_byte": [
        35717120
      ],
      "exit_codes": [
        0
      ]
    }
  ]
}

@harrallo

Copy link
Copy Markdown
Author

multi-thread try.

@harrallo

Copy link
Copy Markdown
Author

/bench

@xHeaven

xHeaven commented Mar 15, 2026

Copy link
Copy Markdown
Member

Challenge over. No more commits from now on, otherwise your submission will be invalidated. Benchmarks will continue.

Also, you should've had a separate PR for a multithreaded attempt. Please forcepush to your latest singlethread commit if you want to receive further benchmarks for this submission.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants