ci: dispatch website rebuild on release-branch doc changes (drop push-based sync) - #7659
Draft
Ma77Ball wants to merge 1 commit into
Draft
ci: dispatch website rebuild on release-branch doc changes (drop push-based sync)#7659Ma77Ball wants to merge 1 commit into
Ma77Ball wants to merge 1 commit into
Conversation
Removes sync-docs-to-site.yml (pushed committed docs into the website) and adds notify-site-rebuild.yml, which dispatches the site's publish-site.yml on staging when release-branch docs change. Aligns with the website's build-time docs sync (apache/incubator-texera-site#72).
Contributor
Automated Reviewer SuggestionsBased on the
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7659 +/- ##
============================================
- Coverage 89.68% 89.67% -0.01%
+ Complexity 4399 4397 -2
============================================
Files 1175 1175
Lines 46979 46979
Branches 5265 5265
============================================
- Hits 42133 42130 -3
- Misses 3082 3083 +1
- Partials 1764 1766 +2
☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Contributor
|
| config | throughput | MB/s | latency | max Δ latest / 7d | |
|---|---|---|---|---|---|
| 🔴 | bs=10 sw=10 sl=64 | 384 | 0.234 | 24,607/34,307/34,307 us | 🔴 -7.9% / 🔴 +109.9% |
| 🔴 | bs=100 sw=10 sl=64 | 759 | 0.463 | 126,000/183,264/183,264 us | 🔴 +28.0% / 🔴 +70.3% |
| ⚪ | bs=1000 sw=10 sl=64 | 884 | 0.54 | 1,129,073/1,177,653/1,177,653 us | ⚪ within ±5% / 🔴 -14.2% |
Baseline details
Latest main 4013ca7 from same runner
| config | metric | PR | latest main | 7d avg | Δ latest | Δ 7d |
|---|---|---|---|---|---|---|
| bs=10 sw=10 sl=64 | throughput | 384 tuples/sec | 417 tuples/sec | 774.89 tuples/sec | -7.9% | -50.4% |
| bs=10 sw=10 sl=64 | MB/s | 0.234 MB/s | 0.254 MB/s | 0.473 MB/s | -7.9% | -50.5% |
| bs=10 sw=10 sl=64 | p50 | 24,607 us | 23,798 us | 12,738 us | +3.4% | +93.2% |
| bs=10 sw=10 sl=64 | p95 | 34,307 us | 35,879 us | 16,348 us | -4.4% | +109.9% |
| bs=10 sw=10 sl=64 | p99 | 34,307 us | 35,879 us | 18,848 us | -4.4% | +82.0% |
| bs=100 sw=10 sl=64 | throughput | 759 tuples/sec | 805 tuples/sec | 1,005 tuples/sec | -5.7% | -24.5% |
| bs=100 sw=10 sl=64 | MB/s | 0.463 MB/s | 0.492 MB/s | 0.613 MB/s | -5.9% | -24.5% |
| bs=100 sw=10 sl=64 | p50 | 126,000 us | 120,286 us | 100,970 us | +4.8% | +24.8% |
| bs=100 sw=10 sl=64 | p95 | 183,264 us | 143,136 us | 107,605 us | +28.0% | +70.3% |
| bs=100 sw=10 sl=64 | p99 | 183,264 us | 143,136 us | 116,429 us | +28.0% | +57.4% |
| bs=1000 sw=10 sl=64 | throughput | 884 tuples/sec | 900 tuples/sec | 1,030 tuples/sec | -1.8% | -14.2% |
| bs=1000 sw=10 sl=64 | MB/s | 0.54 MB/s | 0.549 MB/s | 0.629 MB/s | -1.6% | -14.1% |
| bs=1000 sw=10 sl=64 | p50 | 1,129,073 us | 1,106,750 us | 991,433 us | +2.0% | +13.9% |
| bs=1000 sw=10 sl=64 | p95 | 1,177,653 us | 1,218,730 us | 1,036,668 us | -3.4% | +13.6% |
| bs=1000 sw=10 sl=64 | p99 | 1,177,653 us | 1,218,730 us | 1,070,470 us | -3.4% | +10.0% |
Raw CSV
config_idx,batch_size,schema_width,string_len,num_batches,total_ms,total_tuples,total_bytes,tuples_per_sec,mb_per_sec,lat_p50_us,lat_p95_us,lat_p99_us
0,10,10,64,20,521.09,200,128000,384,0.234,24607.44,34306.50,34306.50
1,100,10,64,20,2634.00,2000,1280000,759,0.463,126000.49,183264.17,183264.17
2,1000,10,64,20,22616.13,20000,12800000,884,0.540,1129072.56,1177653.04,1177653.04
This was referenced Aug 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this PR?
Aligns this repo's CI with the website's new build-time docs sync (merged in apache/incubator-texera-site#72), which pulls each release's
docs/folder straight from this repo'srelease/vX.Ybranches while building the Hugo site.sync-docs-to-site.yml. It pushed committed copies ofdocs/intoapache/incubator-texera-site(content/docs/latest/) on every push tomain. The site no longer stores docs in-tree, so this push-based sync is obsolete and would fight the build-time pull.notify-site-rebuild.yml. When docs change on arelease/v*branch, it dispatches the site's existing "Build & Publish Hugo site" workflow (publish-site.yml) onstagingviaworkflow_dispatch, so the edit becomes visible without waiting for an unrelated site commit. The ping is best-effort (continue-on-error) and never fails this repo's CI.Why not the earlier approach (#6948)?
#6948 was written for an earlier site design that added a
repository_dispatch: docs-updatedtrigger to the site workflow. The design that actually shipped (#72) does not listen forrepository_dispatch; it exposesworkflow_dispatch. This PR targets what shipped, and needs no change on the site side. #6948 can be closed in favor of this.Configuration required
Add a repository secret
SITE_DISPATCH_TOKENwith Actions: write onapache/incubator-texera-site(a fine-grained PAT scoped to that repo, or a classic PAT with theworkflowscope). The oldSITE_SYNC_TOKEN(Contents: write) used by the removed workflow can be retired.Notes / follow-ups
stagingbecause Issue #71: Refactored DataReader and DataWriter #72 currently lives on the site'sstagingbranch. Once Issue #71: Refactored DataReader and DataWriter #72 reaches the site'smain, add a second dispatch on--ref main(production).scripts/sync-docs.shhas an explicitVERSIONSlist (release/v1.1,release/v1.2); a brand-newrelease/vX.Ystill needs to be added there before it publishes.Was this PR authored or co-authored using generative AI tooling?
Co-authored with Claude Opus 4.8 in compliance with ASF policy.