Skip to content

Commit 05ebded

Browse files
authored
TCP first message loss (#699)
* Refs #20508: Add wait_for_logical_port_negotiation_ms to transport descriptor Signed-off-by: Jesus Perez <[email protected]> * Refs #20508: Reuse unused transport descriptor tcp_negotiation_timeout Signed-off-by: Jesus Perez <[email protected]> --------- Signed-off-by: Jesus Perez <[email protected]>
1 parent b95cc4a commit 05ebded

File tree

3 files changed

+11
-0
lines changed

3 files changed

+11
-0
lines changed

docs/03-exports/aliases-api.include

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -632,6 +632,7 @@
632632
.. |TCPTransportDescriptor::keep_alive_thread-api| replace:: :cpp:var:`keep_alive_thread<eprosima::fastdds::rtps::TCPTransportDescriptor::keep_alive_thread>`
633633
.. |TCPTransportDescriptor::accept_thread-api| replace:: :cpp:var:`accept_thread<eprosima::fastdds::rtps::TCPTransportDescriptor::accept_thread>`
634634
.. |TCPTransportDescriptor::tls_config-api| replace:: :cpp:var:`tls_config<eprosima::fastdds::rtps::TCPTransportDescriptor::tls_config>`
635+
.. |TCPTransportDescriptor::tcp_negotiation_timeout-api| replace:: :cpp:var:`tcp_negotiation_timeout<eprosima::fastdds::rtps::TCPTransportDescriptor::tcp_negotiation_timeout>`
635636

636637
.. |TCPTransportDescriptor::TLSConfig-api| replace:: :cpp:struct:`TLSConfig<eprosima::fastdds::rtps::TCPTransportDescriptor::TLSConfig>`
637638
.. |TCPTransportDescriptor::TLSConfig::add_verify_mode-api| replace:: :cpp:func:`add_verify_mode()<eprosima::fastdds::rtps::TCPTransportDescriptor::TLSConfig::add_verify_mode>`

docs/fastdds/transport/tcp/tcp.rst

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,13 @@ The following table describes the common data members for both TCPv4 and TCPv6.
132132
- |ThreadSettings|
133133
-
134134
- |ThreadSettings| for the threads processing incoming TCP connection requests.
135+
* - |TCPTransportDescriptor::tcp_negotiation_timeout-api|
136+
- ``uint32_t``
137+
- 0
138+
- Time to wait for logical port negotiation (in ms). If a logical port is under negotiation, it waits for the
139+
negotiation to finish up to this timeout before trying to send a message to that port. Setting this option to
140+
non-zero values increases the discovery time. Setting it to zero means no wait but could lead to loss of first
141+
messages.
135142

136143
.. warning::
137144

docs/fastdds/xml_configuration/transports.rst

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,9 @@ A more detailed explanation of each of these elements can be found in :ref:`comm
111111
| ``<enable_tcp_nodelay>`` | Socket option for disabling the Nagle |br| | ``bool`` | ``false``|
112112
| | algorithm. (**TCP only**). | | |
113113
+-------------------------------+----------------------------------------------------+----------------------+----------+
114+
| ``<tcp_negotiation_timeout>`` | Time to wait for logical port negotiation (in ms) | ``uint32_t`` | ``0`` |
115+
| | |br| (**TCP only**). | | |
116+
+-------------------------------+----------------------------------------------------+----------------------+----------+
114117
| ``<segment_size>`` | Size (in bytes) of the shared-memory segment. |br| | ``uint32_t`` | 262144 |
115118
| | (Optional, **SHM only**). | | |
116119
+-------------------------------+----------------------------------------------------+----------------------+----------+

0 commit comments

Comments
 (0)