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.

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

[21188] Example Refactor: Request-Reply #5014

Merged
merged 46 commits into from
Jul 16, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
46 commits
Select commit Hold shift + click to select a range
1f80c01
Refs #21188: Update header guards in other examples
EduPonz Jun 18, 2024
59b1233
Refs #21188: CLIParser and example skeleton
EduPonz Jun 19, 2024
ea54cea
Refs #21188: Add IDL and type support
EduPonz Jun 27, 2024
67ec669
Refs #21188: Add Server initialization
EduPonz Jun 27, 2024
30bbd9c
Refs #21188: Add Server on_subscription_matched
EduPonz Jun 27, 2024
f802cca
Refs #21188: Add Client initialization
EduPonz Jun 27, 2024
7829aed
Refs #21188: Add Client on_publication_matched
EduPonz Jun 27, 2024
d599241
Refs #21188: Uncrustify
EduPonz Jun 28, 2024
7c51b2d
Refs #21188: Code grouping
EduPonz Jun 28, 2024
ca43e5c
Refs #21188: Refactor matched status and create a utils header
EduPonz Jun 28, 2024
155efc4
Refs #21188: Client waits until a server is fully matched
EduPonz Jun 28, 2024
8b1075e
Refs #21188: Server runs until it is stopped
EduPonz Jun 28, 2024
5815b8e
Refs #21188: Client sends requests which are put in a queue
EduPonz Jun 28, 2024
d4b30f3
Refs #21188: Rename reply z to result
EduPonz Jun 29, 2024
a4bf006
Refs #21188: Server sends reply and client waits for it
EduPonz Jun 29, 2024
6473037
Refs #21188: Move type support to dedicated directory
EduPonz Jun 29, 2024
4b52346
Refs #21188: Server reply history cleanup when possible
EduPonz Jun 29, 2024
6a5f564
Refs #21188: Client uses CFT to only receive replies to its requests
EduPonz Jun 29, 2024
86e84c2
Refs #21188: Client waits before sending request
EduPonz Jun 29, 2024
1250a32
Refs #21188: Add logging utils
EduPonz Jun 29, 2024
ed7ac41
Refs #21188: Client SIGINT exit before any server has matched
EduPonz Jun 29, 2024
4089d76
Refs #21188: Add debug logging
EduPonz Jun 29, 2024
a412a17
Refs #21188: Add request ID to log messages
EduPonz Jun 29, 2024
c799e6c
Refs #21188: Add small TODO for follow-up PR to fulfil
EduPonz Jul 1, 2024
d828255
Refs #21188: Add context to log messages
EduPonz Jul 1, 2024
e8b09f8
Refs #21188: Add tests
EduPonz Jul 1, 2024
7a3f1cc
Refs #21188: Add README.md
EduPonz Jul 1, 2024
e3baa96
Refs #21188: Add versions
EduPonz Jul 1, 2024
412dfe5
Refs #21188: Remove old example
EduPonz Jul 1, 2024
f34ec0f
Refs #21188: Address minor Eliana's comments
EduPonz Jul 9, 2024
d9cf2f5
Refs #21188: Remove operation param from CLI. Client always requests +
EduPonz Jul 10, 2024
f27a9d8
Refs #21188: Add space between doxygen @param and direction specifier
EduPonz Jul 10, 2024
83f2faf
Refs #21188: Client sends 1 request per operation and waits for all r…
EduPonz Jul 10, 2024
d13e20f
Refs #21188: Fix data race on reader filter removal
EduPonz Jul 10, 2024
e327ed7
Refs #21188: Unmatch participant on participant removed
EduPonz Jul 10, 2024
6165d9f
Refs #21188: Add DomainParticipantFactory::get_participant_extended_q…
EduPonz Jul 10, 2024
ec7d224
Refs #21188: Server ignores non-client participants
EduPonz Jul 10, 2024
61869f6
Refs #21188: Client ignores non-server participants
EduPonz Jul 10, 2024
3b769aa
Refs #21188: Improve client log
EduPonz Jul 10, 2024
a82bccc
Refs #21188: Update tests
EduPonz Jul 10, 2024
2492375
Refs #21188: Increase timeouts for CI testing
EduPonz Jul 11, 2024
c0f6687
Refs #21188: Adjust for change in write return type
EduPonz Jul 12, 2024
a5898aa
Refs #21188: Fix data race between requests_status_ init and update
EduPonz Jul 12, 2024
6a51220
Refs #21188: Take timeouts back to 3s
EduPonz Jul 12, 2024
a39e041
Refs #21188: Remove compose name from clients to cope with different …
EduPonz Jul 12, 2024
130a911
Refs #21188: Apply Eliana's suggestions
EduPonz Jul 15, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions examples/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ add_subdirectory(cpp/dds)
add_subdirectory(cpp/delivery_mechanisms)
add_subdirectory(cpp/discovery_server)
add_subdirectory(cpp/hello_world)
add_subdirectory(cpp/request_reply)
add_subdirectory(cpp/rtps)
add_subdirectory(cpp/xtypes)

Expand Down
6 changes: 3 additions & 3 deletions examples/cpp/content_filter/CLIParser.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -12,16 +12,16 @@
// See the License for the specific language governing permissions and
// limitations under the License.

#ifndef FASTDDS_EXAMPLES_CPP_CONTENT_FILTER__CLIPARSER_HPP
#define FASTDDS_EXAMPLES_CPP_CONTENT_FILTER__CLIPARSER_HPP

#include <csignal>
#include <cstdlib>
#include <iostream>
#include <thread>

#include <fastdds/dds/log/Log.hpp>

#ifndef FASTDDS_EXAMPLES_CPP_CONTENT_FILTER__CLIPARSER_HPP
#define FASTDDS_EXAMPLES_CPP_CONTENT_FILTER__CLIPARSER_HPP

namespace eprosima {
namespace fastdds {
namespace examples {
Expand Down
14 changes: 14 additions & 0 deletions examples/cpp/content_filter/CustomContentFilter.hpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,17 @@
// Copyright 2024 Proyectos y Sistemas de Mantenimiento SL (eProsima).
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

#ifndef FASTDDS_EXAMPLES_CPP_CONTENT_FILTER__CUSTOMCONTENTFILTER_HPP
#define FASTDDS_EXAMPLES_CPP_CONTENT_FILTER__CUSTOMCONTENTFILTER_HPP

Expand Down
14 changes: 14 additions & 0 deletions examples/cpp/content_filter/CustomContentFilterFactory.hpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,17 @@
// Copyright 2024 Proyectos y Sistemas de Mantenimiento SL (eProsima).
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

#ifndef FASTDDS_EXAMPLES_CPP_CONTENT_FILTER__CUSTOMCONTENTFILTERFACTORY_HPP
#define FASTDDS_EXAMPLES_CPP_CONTENT_FILTER__CUSTOMCONTENTFILTERFACTORY_HPP

Expand Down
1 change: 0 additions & 1 deletion examples/cpp/dds/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ add_subdirectory(HelloWorldExampleDataSharing)
add_subdirectory(HelloWorldExampleSharedMem)
add_subdirectory(HelloWorldExampleTCP)
add_subdirectory(Keys)
add_subdirectory(RequestReplyExample)
add_subdirectory(StaticHelloWorldExample)
add_subdirectory(WriterLoansExample)
add_subdirectory(ZeroCopyExample)
71 changes: 0 additions & 71 deletions examples/cpp/dds/RequestReplyExample/CMakeLists.txt

This file was deleted.

Loading
Loading