moxml 0.5.156 → 0.5.157
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/.github/workflows/performance-release.yml +17 -0
- data/benchmark/README.md +12 -4
- data/benchmark/scoreboard.rb +13 -7
- data/lib/moxml/version.rb +1 -1
- metadata +1 -1
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA256:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: 87300d8760e99b65ddda1dd2163c09eeef602885700b8bf7159ee8cb06f708f5
|
|
4
|
+
data.tar.gz: 2bcf6f9deb2fd3bef73234ca9d2e9cac11771efc37e8339ecd2cbc82a2a9cb15
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: bfa9a5c9582fc99bae9aed8888c653ce3bdca81a3a5a3b32ed5be598eea5f0900793d9ef1f453cf8185a6838912a4f6393ae43038f1db2e6a9d1ca3a99c34dd6
|
|
7
|
+
data.tar.gz: e07cd31013b48ec143ef5840097c32cba6f00dff10141ca6f2e6371c98bde6ab98bdf920dc1e8701d5d2bea26d961d09d5c35d3229864beffa26e5066dc98e7c
|
|
@@ -72,8 +72,25 @@ jobs:
|
|
|
72
72
|
- name: Measure the scoreboard
|
|
73
73
|
run: |
|
|
74
74
|
SCORE_MARKDOWN=/tmp/section.md \
|
|
75
|
+
SCORE_JSON=/tmp/measured-scoreboard.json \
|
|
76
|
+
SCORE_PUBLISH=1 \
|
|
75
77
|
bundle exec ruby benchmark/scoreboard.rb
|
|
76
78
|
|
|
79
|
+
- name: Upload the measured scoreboard
|
|
80
|
+
if: always()
|
|
81
|
+
uses: actions/upload-artifact@v4
|
|
82
|
+
with:
|
|
83
|
+
name: scoreboard-${{ github.event.workflow_run.id || 'dispatch' }}
|
|
84
|
+
# The measurement behind the published notes section plus the
|
|
85
|
+
# regenerated baseline + blocks: the data commit back to main
|
|
86
|
+
# rides a PR from these files (never a direct push).
|
|
87
|
+
path: |
|
|
88
|
+
/tmp/measured-scoreboard.json
|
|
89
|
+
benchmark/scoreboard.json
|
|
90
|
+
docs/_pages/performance.adoc
|
|
91
|
+
benchmark/README.md
|
|
92
|
+
if-no-files-found: ignore
|
|
93
|
+
|
|
77
94
|
- name: Append the scoreboard to the release notes of the measured version
|
|
78
95
|
env:
|
|
79
96
|
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
data/benchmark/README.md
CHANGED
|
@@ -27,7 +27,7 @@ with raw nokogiri on the same operation?**
|
|
|
27
27
|
| --- | --------- | ---- |
|
|
28
28
|
| `parse` | `Nokogiri::XML` vs `ctx.parse` on the catalog document | parity |
|
|
29
29
|
| `serialize` | `doc.to_xml` | parity |
|
|
30
|
-
| `xpath //field (count)` | node-set cardinality, nothing materialized |
|
|
30
|
+
| `xpath //field (count)` | node-set cardinality, nothing materialized | parity |
|
|
31
31
|
| `xpath //field (each)` | iterate every result (wrapper materialization) | structural |
|
|
32
32
|
| `attr pairs listing` | collect `[name, value]` for every record | parity |
|
|
33
33
|
| `build add_element` | create + attach 300 records x 4 fields with attrs and text | structural |
|
|
@@ -66,9 +66,17 @@ SCORE_LOCK=1 SCORE_BASELINE=benchmark/scoreboard.json \
|
|
|
66
66
|
Rows with a tracked upstream regression are logged as paused and do not
|
|
67
67
|
fail the lock (`KNOWN_REGRESSIONS` in `scoreboard.rb`); they stay
|
|
68
68
|
measured and published, and the entry is removed when its issue lands.
|
|
69
|
-
The current entries are the
|
|
70
|
-
(`walk+read (
|
|
71
|
-
|
|
69
|
+
The current entries are the two cold crossing-bound walk rows
|
|
70
|
+
(`walk+read (cold)`, `children recursion (cold)`) under
|
|
71
|
+
leptris-ruby#527 (the per-element crossings of the 1.9.380+ line —
|
|
72
|
+
facade-side, not evaluator-side: the engine's own eval and walk
|
|
73
|
+
measured flat across 1.9.374→1.9.380). `walk+read (warm)` recovered
|
|
74
|
+
above parity (1.15-1.17x on the release runs) and left the pause
|
|
75
|
+
list — a future dip now fails the lock again. The count row's
|
|
76
|
+
0.70x reading that opened #527 was the adapter minting every row
|
|
77
|
+
before the caller asked (fixed by the deferred bulk mint); it
|
|
78
|
+
measures 3.50x on the released tree and is promoted back to a
|
|
79
|
+
parity row.
|
|
72
80
|
|
|
73
81
|
A smoke run on a loaded machine:
|
|
74
82
|
|
data/benchmark/scoreboard.rb
CHANGED
|
@@ -51,7 +51,6 @@ module Scoreboard
|
|
|
51
51
|
# the regressed value and the lock signal is lost. Remove an entry
|
|
52
52
|
# when its issue lands.
|
|
53
53
|
KNOWN_REGRESSIONS = {
|
|
54
|
-
"walk+read (warm)" => "leptris-ruby#527 (1.9.380 crossing regression)",
|
|
55
54
|
"walk+read (cold)" => "leptris-ruby#527 (1.9.380 crossing regression)",
|
|
56
55
|
"children recursion (cold)" =>
|
|
57
56
|
"leptris-ruby#527 (1.9.380 crossing regression)",
|
|
@@ -191,11 +190,11 @@ module Scoreboard
|
|
|
191
190
|
# opened leptris-ruby#527 was the adapter minting every row
|
|
192
191
|
# before the caller asked (leptris-ruby#527 verification:
|
|
193
192
|
# engine eval + count is ~0.75us and flat across the
|
|
194
|
-
# 1.9.374->1.9.380 line). The bulk mint is
|
|
195
|
-
# size answers from the result handle
|
|
196
|
-
#
|
|
197
|
-
#
|
|
198
|
-
"xpath //field (count)" => [:
|
|
193
|
+
# 1.9.374->1.9.380 line). The bulk mint is deferred (#444
|
|
194
|
+
# LazyBulkRows), so size answers from the result handle:
|
|
195
|
+
# the released tree measures 3.50x on both release runs
|
|
196
|
+
# (v0.5.155, v0.5.156) — the promotion condition above.
|
|
197
|
+
"xpath //field (count)" => [:parity,
|
|
199
198
|
-> { baseline_doc.xpath("//field").size },
|
|
200
199
|
-> { moxml_doc.xpath("//field").size }],
|
|
201
200
|
# Materialized results: what a consumer iterating the set pays.
|
|
@@ -421,7 +420,14 @@ module Scoreboard
|
|
|
421
420
|
when "parity"
|
|
422
421
|
row["ratio"] >= PARITY_FLOOR ? "met" : "open (below parity)"
|
|
423
422
|
when "structural"
|
|
424
|
-
|
|
423
|
+
status =
|
|
424
|
+
if row["ratio"] >= PARITY_FLOOR
|
|
425
|
+
"tracked (at parity)"
|
|
426
|
+
else
|
|
427
|
+
"tracked (below parity)"
|
|
428
|
+
end
|
|
429
|
+
ref = KNOWN_REGRESSIONS[row["row"]]
|
|
430
|
+
ref ? "#{status}, paused: #{ref}" : status
|
|
425
431
|
else
|
|
426
432
|
"informational"
|
|
427
433
|
end
|
data/lib/moxml/version.rb
CHANGED