Skip to content

Commit 83d1bcd

Browse files
dfa1claude
andcommitted
fix(ci): align bench script with CI, simplify workflow
bench now runs full package build (no -pl performance -am) matching CI behavior. Workflow calls ./bench directly — no separate build step. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 01b75fd commit 83d1bcd

3 files changed

Lines changed: 3 additions & 15 deletions

File tree

.github/workflows/benchmark.yml

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -37,18 +37,8 @@ jobs:
3737
unzip -j flatc.zip flatc
3838
sudo install -m 755 flatc /usr/local/bin/flatc
3939
40-
- name: Build
41-
run: ./mvnw package -DskipTests
42-
4340
- name: Run benchmarks
44-
run: |
45-
java \
46-
--add-opens java.base/java.nio=ALL-UNNAMED \
47-
--enable-native-access=ALL-UNNAMED \
48-
--sun-misc-unsafe-memory-access=allow \
49-
-jar performance/target/benchmarks.jar \
50-
-rf json \
51-
-rff performance/target/benchmark-result.json
41+
run: ./bench
5242

5343
- name: Store benchmark results
5444
uses: benchmark-action/github-action-benchmark@v1

CLAUDE.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,8 @@ Never use `mvn install` or `./mvwn install`.
4949
# Run all benchmarks
5050
./bench
5151

52-
# Run specific benchmark class
52+
# Run specific benchmark class or method (always use ClassName.methodName filter)
5353
./bench RustVsJavaReadBenchmark
54-
55-
# Run specific benchmark method (always use ClassName.methodName filter)
5654
./bench RustVsJavaReadBenchmark.javaReadVolume
5755
```
5856

bench

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#!/usr/bin/env bash
22
set -euo pipefail
3-
./mvnw package -pl performance -am -DskipTests -q
3+
./mvnw package -DskipTests
44
java \
55
--add-opens java.base/java.nio=ALL-UNNAMED \
66
--enable-native-access=ALL-UNNAMED \

0 commit comments

Comments
 (0)