feat(libp2p): Add more columns to HANDLE_METADATA/STATUS #661
Conversation
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
…connection direction
mattevans
approved these changes
Sep 29, 2025
…direction column The distributed tables `libp2p_handle_status` and `libp2p_handle_metadata` are dropped and recreated. This is necessary to correctly propagate the newly added `direction` column from the local tables to the distributed tables, ensuring data consistency across clusters.
Adds a `direction` column to `libp2p_handle_status` and `libp2p_handle_metadata` tables (both local and distributed) in ClickHouse migrations. This is necessary to track whether an RPC request was inbound or outbound. The migration drops and recreates distributed tables to accommodate the schema change.
Savid
reviewed
Oct 1, 2025
deploy/migrations/clickhouse/078_add_direction_to_metadata_status.up.sql
Outdated
Show resolved
Hide resolved
Savid
reviewed
Oct 1, 2025
deploy/migrations/clickhouse/078_add_direction_to_metadata_status.down.sql
Outdated
Show resolved
Hide resolved
Co-authored-by: Andrew Davis <[email protected]> Signed-off-by: Sam Calder-Mason <[email protected]>
Co-authored-by: Andrew Davis <[email protected]> Signed-off-by: Sam Calder-Mason <[email protected]>
…QL file Removes extraneous characters (` ``` `) at the end of the `ALTER TABLE` statement in the ClickHouse migration script to ensure valid SQL execution.
… libp2p metrics This commit introduces two new fields to track more detailed peer RPC information: - `earliest_available_slot` in `Status` for RPC responses. - `custody_group_count` in `Metadata`. This requires updating Vector configuration, ClickHouse migrations (up/down), and the Go protobuf definitions to correctly ingest and store this new data.
samcm
changed the title
mattevans
reviewed
Oct 1, 2025
mattevans
reviewed
Oct 1, 2025
… CustodyGroupCount in HandleMetadata Adds `EarliestAvailableSlot` to `HandleStatus` for both request and response tracing events. Also maps `CustodyGroupCount` from trace event payload to `HandleMetadata`. This enhances visibility into the state and progress of libp2p handling operations.
mattevans
approved these changes
Oct 1, 2025
* master: feat: add support for Tysm consensus client in cannon and sentry
refactor: align struct field spacing for consistent formatting
…status-direction-matty fix: align custodyGroupCount type assertion with protobuf uint64
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
No description provided.