[ 
https://issues.apache.org/jira/browse/PROTON-2308?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463701#comment-17463701
 ] 

ASF GitHub Bot commented on PROTON-2308:
----------------------------------------

gemmellr commented on a change in pull request #346:
URL: https://github.com/apache/qpid-proton/pull/346#discussion_r773774519



##########
File path: cpp/src/link_test.cpp
##########
@@ -23,9 +23,114 @@
 #include <proton/sender_options.hpp>
 #include <proton/receiver_options.hpp>
 #include <proton/container.hpp>
+#include <proton/connection.hpp>
+#include <proton/connection_options.hpp>
+#include <proton/listen_handler.hpp>
+#include <proton/listener.hpp>
+#include <proton/messaging_handler.hpp>
+#include <proton/types.hpp>
+#include <proton/message.hpp>
+#include <proton/target_options.hpp>
+#include <proton/source_options.hpp>
+#include <proton/delivery.hpp>
 
 #include <iostream>
 
+#include <map>
+#include <condition_variable>
+#include <mutex>
+#include <thread>
+
+namespace {
+std::mutex m;
+std::condition_variable cv;
+bool listener_ready = false;
+int listener_port;
+const std::string DNP_SYMBOL = "DNP_SYMBOL";
+const std::string DNP_VALUE = "DNP_VALUE";
+} // namespace

Review comment:
       The supported-dist-modes would be the simplest real noe property (as 
anything legal is allwoed I didnt see much need to use specific values, but it 
would be a representative test). The supported-dist-modes value is noted as 
having a 'multiple' definition, which means it can takes a single symbol or an 
array (not list) of symbols, with null or an empty array both meaning 
not-specified (per section 1.4 Composite Type Representation in the spec).




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [cpp] Add support for setting Dynamic Node Properties
> -----------------------------------------------------
>
>                 Key: PROTON-2308
>                 URL: https://issues.apache.org/jira/browse/PROTON-2308
>             Project: Qpid Proton
>          Issue Type: Improvement
>          Components: cpp-binding
>    Affects Versions: proton-c-0.33.0
>            Reporter: James Henry
>            Assignee: Rakhi Kumari
>            Priority: Major
>              Labels: api-addition
>
> Requesting support for setting the dynamic node properties be added to the 
> source and target options.
> This would allow the setting of termini node properties for senders and 
> receivers.
> Similar to the following request made for Python here: 
> https://issues.apache.org/jira/browse/PROTON-816



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org

Reply via email to