Skip to content

Type hash distribution during discovery (rep2011)#104

Merged
clalancette merged 9 commits intoros2:rollingfrom
emersonknapp:emersonknapp/type-version-hash
Mar 26, 2023
Merged

Type hash distribution during discovery (rep2011)#104
clalancette merged 9 commits intoros2:rollingfrom
emersonknapp:emersonknapp/type-version-hash

Conversation

@emersonknapp
Copy link
Contributor

@emersonknapp emersonknapp commented Feb 23, 2023

Part of ros2/ros2#1159

Depends on ros2/rosidl#729
Depends on ros2/rmw_dds_common#70

Features:

  • Retrieve type version hash from generated code, and use the USER_DATA QoS policy to distribute it for all readers/writers. Format of the USER_DATA uses common utility functions in rmw_dds_common
  • On discovering new endpoints, parse the USER_DATA to set rmw_topic_endpoint_info_t.type_hash

@emersonknapp emersonknapp changed the title [WIP] Type hash distribution during discovery (rep2011) Type hash distribution during discovery (rep2011) Feb 23, 2023
@emersonknapp emersonknapp marked this pull request as ready for review February 23, 2023 23:22
@emersonknapp emersonknapp force-pushed the emersonknapp/type-version-hash branch from 63f81ff to 1c54066 Compare March 16, 2023 02:27
Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two small changes, this otherwise looks good to me.

@emersonknapp
Copy link
Contributor Author

emersonknapp commented Mar 17, 2023

@emersonknapp emersonknapp force-pushed the emersonknapp/type-version-hash branch from 5ff54f7 to 4290374 Compare March 20, 2023 06:07
@emersonknapp
Copy link
Contributor Author

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@emersonknapp emersonknapp force-pushed the emersonknapp/type-version-hash branch from 4290374 to 4efe847 Compare March 22, 2023 05:59
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
@emersonknapp emersonknapp force-pushed the emersonknapp/type-version-hash branch from c1f26b2 to 4e527bd Compare March 24, 2023 17:05
@emersonknapp
Copy link
Contributor Author

emersonknapp commented Mar 24, 2023

Gist: https://gist.githubusercontent.com/emersonknapp/c36ca5985030eccbcaac0d0384236fbd/raw/374f6ef848452efc7e6a2b079129195ece75aeec/ros2.repos
BUILD args: --packages-above-and-dependencies rmw_dds_common
TEST args: --packages-above rmw_dds_common
ROS Distro: rolling

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@clalancette
Copy link
Contributor

CI is in ros2/rmw_dds_common#70 (comment)

@clalancette clalancette merged commit ce5f683 into ros2:rolling Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants