From e73df29cf80723f0e49e09d1f72cff5f1d342744 Mon Sep 17 00:00:00 2001 From: Chris Lalancette Date: Wed, 26 Jun 2024 18:03:55 -0400 Subject: [PATCH] Switch wait_for_all_acked to return OK. (#215) This just keeps upper-layer tests working. Signed-off-by: Chris Lalancette --- rmw_zenoh_cpp/src/rmw_zenoh.cpp | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/rmw_zenoh_cpp/src/rmw_zenoh.cpp b/rmw_zenoh_cpp/src/rmw_zenoh.cpp index 3210da6d..4d2fffcf 100644 --- a/rmw_zenoh_cpp/src/rmw_zenoh.cpp +++ b/rmw_zenoh_cpp/src/rmw_zenoh.cpp @@ -1178,7 +1178,13 @@ rmw_publisher_wait_for_all_acked( { static_cast(publisher); static_cast(wait_timeout); - return RMW_RET_UNSUPPORTED; + + // We are not currently tracking all published data, so we don't know what data is in flight that + // we might have to wait for. Even if we did start tracking it, we don't have insight into the + // TCP stream that Zenoh is managing for us, so we couldn't guarantee this anyway. + // Just lie to the upper layers and say that everything is working as expected. + // We return OK rather than UNSUPPORTED so that certain upper-layer tests continue working. + return RMW_RET_OK; } //==============================================================================