ci: remove sync-docs-to-site workflow (superseded by build-time docs sync) - #7658
Closed
Ma77Ball wants to merge 1 commit into
Closed
ci: remove sync-docs-to-site workflow (superseded by build-time docs sync)#7658Ma77Ball wants to merge 1 commit into
Ma77Ball wants to merge 1 commit into
Conversation
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 #7658 +/- ##
=========================================
Coverage 89.68% 89.68%
Complexity 4399 4399
=========================================
Files 1175 1175
Lines 46979 46979
Branches 5265 5265
=========================================
Hits 42133 42133
Misses 3082 3082
Partials 1764 1764
*This pull request uses carry forward flags. Click here to find out more. ☔ 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 | 402 | 0.245 | 23,003/32,911/32,911 us | 🔴 +5.7% / 🔴 +101.3% |
| ⚪ | bs=100 sw=10 sl=64 | 802 | 0.49 | 123,062/145,769/145,769 us | ⚪ within ±5% / 🔴 +35.5% |
| ⚪ | bs=1000 sw=10 sl=64 | 913 | 0.557 | 1,097,402/1,158,713/1,158,713 us | ⚪ within ±5% / 🔴 +11.8% |
Baseline details
Latest main 4013ca7 from same runner
| config | metric | PR | latest main | 7d avg | Δ latest | Δ 7d |
|---|---|---|---|---|---|---|
| bs=10 sw=10 sl=64 | throughput | 402 tuples/sec | 420 tuples/sec | 774.89 tuples/sec | -4.3% | -48.1% |
| bs=10 sw=10 sl=64 | MB/s | 0.245 MB/s | 0.257 MB/s | 0.473 MB/s | -4.7% | -48.2% |
| bs=10 sw=10 sl=64 | p50 | 23,003 us | 23,288 us | 12,738 us | -1.2% | +80.6% |
| bs=10 sw=10 sl=64 | p95 | 32,911 us | 31,136 us | 16,348 us | +5.7% | +101.3% |
| bs=10 sw=10 sl=64 | p99 | 32,911 us | 31,136 us | 18,848 us | +5.7% | +74.6% |
| bs=100 sw=10 sl=64 | throughput | 802 tuples/sec | 817 tuples/sec | 1,005 tuples/sec | -1.8% | -20.2% |
| bs=100 sw=10 sl=64 | MB/s | 0.49 MB/s | 0.499 MB/s | 0.613 MB/s | -1.8% | -20.1% |
| bs=100 sw=10 sl=64 | p50 | 123,062 us | 120,760 us | 100,970 us | +1.9% | +21.9% |
| bs=100 sw=10 sl=64 | p95 | 145,769 us | 148,494 us | 107,605 us | -1.8% | +35.5% |
| bs=100 sw=10 sl=64 | p99 | 145,769 us | 148,494 us | 116,429 us | -1.8% | +25.2% |
| bs=1000 sw=10 sl=64 | throughput | 913 tuples/sec | 910 tuples/sec | 1,030 tuples/sec | +0.3% | -11.4% |
| bs=1000 sw=10 sl=64 | MB/s | 0.557 MB/s | 0.555 MB/s | 0.629 MB/s | +0.4% | -11.4% |
| bs=1000 sw=10 sl=64 | p50 | 1,097,402 us | 1,095,284 us | 991,433 us | +0.2% | +10.7% |
| bs=1000 sw=10 sl=64 | p95 | 1,158,713 us | 1,135,714 us | 1,036,668 us | +2.0% | +11.8% |
| bs=1000 sw=10 sl=64 | p99 | 1,158,713 us | 1,135,714 us | 1,070,470 us | +2.0% | +8.2% |
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,497.73,200,128000,402,0.245,23003.37,32910.56,32910.56
1,100,10,64,20,2492.27,2000,1280000,802,0.490,123061.68,145768.65,145768.65
2,1000,10,64,20,21899.92,20000,12800000,913,0.557,1097402.46,1158712.84,1158712.84
Contributor
Author
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.
Removes the
sync-docs-to-site.ymlworkflow, which pushed committed copies ofdocs/intoapache/incubator-texera-site(content/docs/latest/) on every push tomain.Why
The website is moving to a build-time docs sync:
apache/incubator-texera-sitenow pulls docs directly from this repo'srelease/vX.Ybranches while building the Hugo site (see the site'sscripts/pull-release-docs.shand updatedpublish-site.yml). Under that modelcontent/docs/latest/is generated at build time and git-ignored, never committed.Keeping this workflow would fight the new setup: it would keep committing docs into
content/docs/latest/on the site repo, which the build-time sync expects to be ignored, build-generated output. Removing it makes the build-time pull the single source of truth.Rollout note
This should land together with the site's build-time sync going live (and with the release CI firing the
docs-updatedrepository_dispatch the new workflow listens for), since this workflow is currently the only thing keeping the site's docs up to date. Once it's removed, theSITE_SYNC_TOKENsecret it used can be retired.