phrocker commented on a change in pull request #614: MINIFICPP-732 - Add 
property to expose librdkafka "debug" values for …
URL: https://github.com/apache/nifi-minifi-cpp/pull/614#discussion_r307729933
 
 

 ##########
 File path: extensions/librdkafka/PublishKafka.cpp
 ##########
 @@ -79,6 +79,7 @@ core::Property PublishKafka::KerberosKeytabPath("Kerberos 
Keytab Path",
 core::Property PublishKafka::MessageKeyField("Message Key Field", "The name of 
a field in the Input Records that should be used as the Key for the Kafka 
message.\n"
                                              "Supports Expression Language: 
true (will be evaluated using flow file attributes)",
                                              "");
+core::Property PublishKafka::DebugContexts("Debug contexts", "A 
comma-separated list of debug contexts to enable. Including: generic, broker, 
topic, metadata, feature, queue, msg, protocol, cgrp, security, fetch, 
interceptor, plugin, consumer, admin, eos, all", "");
 
 Review comment:
   @nghiaxlee I think I'm the long pole here so since my time is so fragmented. 
I created https://issues.apache.org/jira/browse/MINIFICPP-988 as a follow on 
with no release since there seems to be a disagreement about the approach. You 
are free to make a decision in that ticket, but I ask that you evaluate all 
options holistically. 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to