pulsar-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [pulsar] nlu90 commented on a change in pull request #8432: [Issue 8382][Pulsar Function] Enable e2e encryption for Pulsar Function
Date Tue, 03 Nov 2020 19:10:45 GMT

nlu90 commented on a change in pull request #8432:
URL: https://github.com/apache/pulsar/pull/8432#discussion_r516895932



##########
File path: pulsar-functions/proto/src/main/proto/Function.proto
##########
@@ -100,12 +100,38 @@ message ConsumerSpec {
     ReceiverQueueSize receiverQueueSize = 4;
     map<string, string> schemaProperties = 5;
     map<string, string> consumerProperties = 6;
+    CryptoSpec cryptoSpec = 7;
 }
 
 message ProducerSpec {
     int32 maxPendingMessages = 1;
     int32 maxPendingMessagesAcrossPartitions = 2;
     bool useThreadLocalProducers = 3;
+    CryptoSpec cryptoSpec = 4;
+}
+
+message CryptoSpec {
+    enum FailureAction {
+        FAIL = 0;
+
+        DISCARD = 1;
+        CONSUME = 2;
+
+        SEND = 10;

Review comment:
       Reserve some single digit values just in case there're new actions added for the consumer
failure.
   
   Logically it's more clear. FAIL or 0 is the default behavior; 1~9 is for consumer; 10 and
above is for producer.
   




----------------------------------------------------------------
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:
users@infra.apache.org



Mime
View raw message