Skip to content

patch tipset traces parallelization logic for #7107#7110

Merged
LesnyRumcajs merged 5 commits into
execute-tipset-traces-parallelizefrom
hm/patch-execute-tipset-traces-parallelize
May 27, 2026
Merged

patch tipset traces parallelization logic for #7107#7110
LesnyRumcajs merged 5 commits into
execute-tipset-traces-parallelizefrom
hm/patch-execute-tipset-traces-parallelize

Conversation

@hanabi1224

Copy link
Copy Markdown
Contributor

Summary of changes

Changes introduced in this pull request:

Benchmark

# command
oha -n 1000 -c 1 -m POST http://localhost:2345/rpc/v1 -H 'Content-Type: application/json' -D trace_block_call.json

# payload
{
	"method": "trace_block",
	"params": [
		"0x5c4ce8"
	],
	"id": 328,
	"jsonrpc": "2.0"
}

# main
  Success rate:m100.00%   4m    5m    6m    7m    8m       ││0.0314  0.0701  0.1087  0.1474  0.1861  0.2248  0.2635    │
  Total:────────61728.4416 ms──────────────────────────────┘└──────────────────────────────────────────────────────────┘
  Slowest:      263.5058 ms
  Fastest:      31.3651 ms
  Average:      61.7077 ms
  Requests/sec: 16.2000

  Total data:   659.85 MiB
  Size/request: 675.69 KiB
  Size/sec:     10.69 MiB

Response time histogram:
   31.365 ms [1]   |
   54.579 ms [445] |■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■
   77.793 ms [177] |■■■■■■■■■■■■
  101.007 ms [349] |■■■■■■■■■■■■■■■■■■■■■■■■■
  124.221 ms [5]   |
  147.435 ms [8]   |
  170.650 ms [13]  |
  193.864 ms [1]   |
  217.078 ms [0]   |
  240.292 ms [0]   |
  263.506 ms [1]   |

Response time distribution:
  10.00% in 33.7217 ms
  25.00% in 35.3091 ms
  50.00% in 68.3582 ms
  75.00% in 81.4580 ms
  90.00% in 84.6755 ms
  95.00% in 86.3642 ms
  99.00% in 158.7673 ms
  99.90% in 263.5058 ms
  99.99% in 263.5058 ms
  
# PR  
  Success rate: 100.00%30s    40s    50s    60s    70s     ││0.0299  0.0436  0.0573  0.0710  0.0846  0.0983  0.1120    │
  Total:────────41611.7485 ms──────────────────────────────┘└──────────────────────────────────────────────────────────┘
  Slowest:      119.1231 ms
  Fastest:      29.3240 ms
  Average:      41.5943 ms
  Requests/sec: 24.0317

  Total data:   659.85 MiB
  Size/request: 675.69 KiB
  Size/sec:     15.86 MiB

Response time histogram:
   29.324 ms [1]   |
   38.304 ms [502] |■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■
   47.284 ms [303] |■■■■■■■■■■■■■■■■■■■
   56.264 ms [138] |■■■■■■■■
   65.244 ms [11]  |
   74.224 ms [6]   |
   83.203 ms [6]   |
   92.183 ms [14]  |
  101.163 ms [12]  |
  110.143 ms [5]   |
  119.123 ms [2]   |

Response time distribution:
  10.00% in 31.5868 ms
  25.00% in 33.3507 ms
  50.00% in 38.1203 ms
  75.00% in 46.4174 ms
  90.00% in 48.7285 ms
  95.00% in 61.7432 ms
  99.00% in 96.8557 ms
  99.90% in 119.1231 ms
  99.99% in 119.1231 ms

Reference issue to close (if applicable)

Closes

Other information and links

Change checklist

  • I have performed a self-review of my own code,
  • I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
  • I have added tests that prove my fix is effective or that my feature works (if possible),
  • I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Outside contributions

  • I have read and agree to the CONTRIBUTING document.
  • I have read and agree to the AI Policy document. I understand that failure to comply with the guidelines will lead to rejection of the pull request.

@coderabbitai

coderabbitai Bot commented May 26, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 1c0e9624-3bf9-4f95-8934-aa65fdea57b5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hm/patch-execute-tipset-traces-parallelize
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch hm/patch-execute-tipset-traces-parallelize

Comment @coderabbitai help to get the list of available commands and usage tips.

@hanabi1224 hanabi1224 added the RPC requires calibnet RPC checks to run on CI label May 26, 2026
@hanabi1224 hanabi1224 marked this pull request as ready for review May 26, 2026 22:11
@hanabi1224 hanabi1224 requested a review from a team as a code owner May 26, 2026 22:11
@hanabi1224 hanabi1224 requested review from LesnyRumcajs and sudo-shashank and removed request for a team May 26, 2026 22:11
@hanabi1224 hanabi1224 changed the title patch tipset traces parallelization logic patch tipset traces parallelization logic for #7107 May 26, 2026
@LesnyRumcajs LesnyRumcajs force-pushed the execute-tipset-traces-parallelize branch from 999b3ad to 7a7f15a Compare May 27, 2026 08:49
…e' into hm/patch-execute-tipset-traces-parallelize
@LesnyRumcajs LesnyRumcajs merged commit 0e00a75 into execute-tipset-traces-parallelize May 27, 2026
3 checks passed
@LesnyRumcajs LesnyRumcajs deleted the hm/patch-execute-tipset-traces-parallelize branch May 27, 2026 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RPC requires calibnet RPC checks to run on CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants