Skip to content

Commit

Permalink
fix: adopt the fix of ze_querying_subscriber_get options
Browse files Browse the repository at this point in the history
  • Loading branch information
YuanYuYuan authored and Yadunund committed Aug 28, 2024
1 parent 9144455 commit 09ee634
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
8 changes: 4 additions & 4 deletions rmw_zenoh_cpp/src/rmw_zenoh.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1500,10 +1500,10 @@ rmw_create_subscription(
"QueryingSubscriberCallback triggered over %s.",
selector.c_str()
);
z_get_options_t opts = z_get_options_default();
opts.target = Z_QUERY_TARGET_ALL_COMPLETE;
opts.timeout_ms = std::numeric_limits<uint64_t>::max();
opts.consolidation = z_query_consolidation_latest();
ze_querying_subscriber_options_t opts = ze_querying_subscriber_options_default();;
opts.query_timeout_ms = std::numeric_limits<uint64_t>::max();
opts.query_consolidation = z_query_consolidation_latest();
opts.query_accept_replies = ZCU_REPLY_KEYEXPR_ANY;
ze_querying_subscriber_get(
z_loan(std::get<ze_owned_querying_subscriber_t>(sub_data->sub)),
z_keyexpr(selector.c_str()),
Expand Down
4 changes: 2 additions & 2 deletions zenoh_c_vendor/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,8 @@ set(ZENOHC_CARGO_FLAGS "--no-default-features$<SEMICOLON>--features=zenoh/shared
# - https://github.com/eclipse-zenoh/zenoh-c/pull/358 (fix debian packaging issue: https://github.com/jspricke/ros-deb-builder-action/issues/49)
# - https://github.com/eclipse-zenoh/zenoh/pull/1150 (fix deadlock issue https://github.com/ros2/rmw_zenoh/issues/182)
ament_vendor(zenoh_c_vendor
VCS_URL https://github.com/eclipse-zenoh/zenoh-c.git
VCS_VERSION 548ee8dde0f53a58c06e68a2949949b31140c36c
VCS_URL https://github.com/ZettaScaleLabs/zenoh-c
VCS_VERSION d4df47382482b1bc7a5b64ca6879ff800779df7b
CMAKE_ARGS
"-DZENOHC_CARGO_FLAGS=${ZENOHC_CARGO_FLAGS}"
)
Expand Down

0 comments on commit 09ee634

Please sign in to comment.