[PR #953] [MERGED] Add parser benchmarks #824

New Issue

adam · 2025-12-30T01:27:02+01:00

adam commented

2025-12-30 01:27:02 +01:00

📋 Pull Request Information

Original PR: https://github.com/apple/pkl/pull/953
Author: @bioball
Created: 2/13/2025
Status: ✅ Merged
Merged: 2/13/2025
Merged by: @bioball

Base: main ← Head: add-parser-benchmarks

📝 Commits (2)

cf41a5f Change parser benchmarks to parse stdlib and snippet tests
ebcb227 Update bench/src/jmh/java/org/pkl/core/parser/ParserBenchmark.java

📊 Changes

4 files changed (+53 additions, -49 deletions)

View changed files

📝 bench/bench.gradle.kts (+1 -1)
📝 bench/gradle.lockfile (+14 -14)
📝 bench/src/jmh/java/org/pkl/core/parser/ParserBenchmark.java (+34 -31)
📝 pkl-commons-test/src/main/kotlin/org/pkl/commons/test/FileTestUtils.kt (+4 -3)

📄 Description

The current benchmark is not too useful now that we have a linear parser.

This changes the benchmarks to parse the standard library, and also the language snippet tests.

Sample output:

Benchmark                           Mode  Cnt    Score    Error  Units
ParserBenchmark.parseSnippetTests  thrpt    5   52.435 ±  2.514  ops/s
ParserBenchmark.parseStdlib        thrpt    5  586.766 ± 95.770  ops/s

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/apple/pkl/pull/953 **Author:** [@bioball](https://github.com/bioball) **Created:** 2/13/2025 **Status:** ✅ Merged **Merged:** 2/13/2025 **Merged by:** [@bioball](https://github.com/bioball) **Base:** `main` ← **Head:** `add-parser-benchmarks` --- ### 📝 Commits (2) - [`cf41a5f`](https://github.com/apple/pkl/commit/cf41a5f2b12f143a3fef32b18108b742d84bd9d8) Change parser benchmarks to parse stdlib and snippet tests - [`ebcb227`](https://github.com/apple/pkl/commit/ebcb227185152031067a856a0eb312fbeb0735bc) Update bench/src/jmh/java/org/pkl/core/parser/ParserBenchmark.java ### 📊 Changes **4 files changed** (+53 additions, -49 deletions) <details> <summary>View changed files</summary> 📝 `bench/bench.gradle.kts` (+1 -1) 📝 `bench/gradle.lockfile` (+14 -14) 📝 `bench/src/jmh/java/org/pkl/core/parser/ParserBenchmark.java` (+34 -31) 📝 `pkl-commons-test/src/main/kotlin/org/pkl/commons/test/FileTestUtils.kt` (+4 -3) </details> ### 📄 Description The current benchmark is not too useful now that we have a linear parser. This changes the benchmarks to parse the standard library, and also the language snippet tests. Sample output: ``` Benchmark Mode Cnt Score Error Units ParserBenchmark.parseSnippetTests thrpt 5 52.435 ± 2.514 ops/s ParserBenchmark.parseStdlib thrpt 5 586.766 ± 95.770 ops/s ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

adam added the pull-request label 2025-12-30 01:27:02 +01:00

adam closed this issue

2025-12-30 01:27:02 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/pkl#824