[go: up one dir, main page]

DAL: add batch validation profiling

This simple MR aims to bring some more profiling to the batch validation.

In addition, it fixes the missing dal node profiling files retrieval.

Manually testing the MR

Not really needed but one can:

  • build with the PPX profiling enabled,
  • run a tezt experiment with profiling and artifact retrieval enabled,
  • Look for the added events
 $ cat TEZT-CLOUD-ARTIFACTS/baker-1/baker-dal-node-1/profiling/gossipsub_profiling.txt| grep triage
  triage ....................................................................... 1            0.957ms 100%
  triage ....................................................................... 1            0.324ms 100%
 […]
$ cat TEZT-CLOUD-ARTIFACTS/baker-1/baker-dal-node-1/profiling/gossipsub_profiling.txt| grep batch 
    apply_batch_event .......................................................... 2           54.572ms 274%
    apply_batch_event .......................................................... 2           27.267ms 326%
[…]

Checklist

  • Document the interface of any function added or modified (see the coding guidelines)
  • Document any change to the user interface, including configuration parameters (see node configuration)
  • Provide automatic testing (see the testing guide).
  • For new features and bug fixes, add an item in the appropriate changelog (docs/protocols/alpha.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR

Merge request reports

Loading