Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add matching subscribers #853

Merged
merged 3 commits into from
Jan 17, 2025
Merged

Add matching subscribers #853

merged 3 commits into from
Jan 17, 2025

Conversation

sashacmc
Copy link
Member

@sashacmc sashacmc commented Jan 8, 2025 •
edited
Loading

Add matching subscribers support.

New API methods:

  • z_publisher_declare_matching_listener
  • z_publisher_declare_background_matching_listener
  • z_publisher_get_matching_status

Test:

  • z_api_matching_test

Example:

  • z_pub with -a key

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 8, 2025

PR missing one of the required labels: {'breaking-change', 'enhancement', 'dependencies', 'documentation', 'bug', 'new feature', 'internal'}

@sashacmc sashacmc added the new feature Something new is needed label Jan 8, 2025
@sashacmc sashacmc force-pushed the add_matching_subscribers branch 9 times, most recently from df86461 to ef5b7a6 Compare January 15, 2025 16:09
@sashacmc sashacmc force-pushed the add_matching_subscribers branch 10 times, most recently from 1131168 to 7afad46 Compare January 16, 2025 13:41
@sashacmc sashacmc force-pushed the add_matching_subscribers branch from 02f2b09 to 0af4661 Compare January 16, 2025 15:01
@sashacmc sashacmc force-pushed the add_matching_subscribers branch 3 times, most recently from ad9df62 to 019e0bb Compare January 16, 2025 16:26
@sashacmc sashacmc force-pushed the add_matching_subscribers branch from 019e0bb to ce01ea0 Compare January 16, 2025 16:36
@sashacmc sashacmc marked this pull request as ready for review January 16, 2025 16:46
@sashacmc sashacmc requested a review from jean-roland January 16, 2025 16:46
@milyin milyin merged commit 6385353 into eclipse-zenoh:main Jan 17, 2025
66 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jean-roland jean-roland jean-roland approved these changes

Assignees
No one assigned
Labels
new feature Something new is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sashacmc @jean-roland @milyin