[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332913435
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
+
+![Delayed Message Delivery](assets/message_delay.png)
+
+broker save message without any check. when consumer consume message, if 
message is delayed type, message will add to DelayedDeliveryTracker. 
subscription will check and get timeout messages from DelayedDeliveryTracker.
+
+### broker 
+Delayed Message Delivery is enabled by default on broker conf as below
+```
+# Whether to enable the delayed delivery for messages.
+# If disabled, messages will be immediately delivered and there will
+# be no tracking overhead.
+delayedDeliveryEnabled=true
+
+# Control the tick time for when retrying on delayed delivery,
+# affecting the accuracy of the delivery time compared to the scheduled time.
+# Default is 1 second.
+delayedDeliveryTickTimeMillis=1000
+```
+
+### producer 
+Here's an example for Delayed message producer in java:
 
 Review comment:
   @Anonymitaet done


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332912987
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
+
+![Delayed Message Delivery](assets/message_delay.png)
+
+broker save message without any check. when consumer consume message, if 
message is delayed type, message will add to DelayedDeliveryTracker. 
subscription will check and get timeout messages from DelayedDeliveryTracker.
+
+### broker 
+Delayed Message Delivery is enabled by default on broker conf as below
+```
+# Whether to enable the delayed delivery for messages.
+# If disabled, messages will be immediately delivered and there will
+# be no tracking overhead.
+delayedDeliveryEnabled=true
+
+# Control the tick time for when retrying on delayed delivery,
+# affecting the accuracy of the delivery time compared to the scheduled time.
+# Default is 1 second.
+delayedDeliveryTickTimeMillis=1000
+```
+
+### producer 
 
 Review comment:
   done @Anonymitaet 


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332912201
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
+
+![Delayed Message Delivery](assets/message_delay.png)
+
+broker save message without any check. when consumer consume message, if 
message is delayed type, message will add to DelayedDeliveryTracker. 
subscription will check and get timeout messages from DelayedDeliveryTracker.
+
+### broker 
+Delayed Message Delivery is enabled by default on broker conf as below
+```
+# Whether to enable the delayed delivery for messages.
+# If disabled, messages will be immediately delivered and there will
+# be no tracking overhead.
+delayedDeliveryEnabled=true
+
+# Control the tick time for when retrying on delayed delivery,
 
 Review comment:
   @Anonymitaet done


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332911640
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
+
+![Delayed Message Delivery](assets/message_delay.png)
+
+broker save message without any check. when consumer consume message, if 
message is delayed type, message will add to DelayedDeliveryTracker. 
subscription will check and get timeout messages from DelayedDeliveryTracker.
+
+### broker 
+Delayed Message Delivery is enabled by default on broker conf as below
+```
+# Whether to enable the delayed delivery for messages.
+# If disabled, messages will be immediately delivered and there will
+# be no tracking overhead.
 
 Review comment:
   done @Anonymitaet 


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332911387
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
+
+![Delayed Message Delivery](assets/message_delay.png)
+
+broker save message without any check. when consumer consume message, if 
message is delayed type, message will add to DelayedDeliveryTracker. 
subscription will check and get timeout messages from DelayedDeliveryTracker.
+
+### broker 
+Delayed Message Delivery is enabled by default on broker conf as below
+```
+# Whether to enable the delayed delivery for messages.
+# If disabled, messages will be immediately delivered and there will
 
 Review comment:
   @Anonymitaet done


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332910988
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
+
+![Delayed Message Delivery](assets/message_delay.png)
+
+broker save message without any check. when consumer consume message, if 
message is delayed type, message will add to DelayedDeliveryTracker. 
subscription will check and get timeout messages from DelayedDeliveryTracker.
+
+### broker 
+Delayed Message Delivery is enabled by default on broker conf as below
 
 Review comment:
   @Anonymitaet done


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332910309
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
+
+![Delayed Message Delivery](assets/message_delay.png)
+
+broker save message without any check. when consumer consume message, if 
message is delayed type, message will add to DelayedDeliveryTracker. 
subscription will check and get timeout messages from DelayedDeliveryTracker.
+
+### broker 
 
 Review comment:
   done @Anonymitaet 


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332910041
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
 
 Review comment:
   done @Anonymitaet 


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332905715
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
+
+![Delayed Message Delivery](assets/message_delay.png)
+
+broker save message without any check. when consumer consume message, if 
message is delayed type, message will add to DelayedDeliveryTracker. 
subscription will check and get timeout messages from DelayedDeliveryTracker.
 
 Review comment:
   @Anonymitaet done


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332904033
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
 Review comment:
   @Anonymitaet  done


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-09 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r332902612
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memory after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
 Review comment:
   @codelipenghui done


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-09-30 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r329529524
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memoey after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
+
+![Delayed Message Delivery](assets/message_delay.png)
+
+broker save message without any check. when consumer consume message, if 
message is delayed type, message will add to DelayedDeliveryTracker. 
subscription will check and get timeout messages from DelayedDeliveryTracker.
+
+### broker 
+Delayed Message Delivery is enabled by default on broker conf as below
+```
+# Whether to enable the delayed delivery for messages.
+# If disabled, messages will be immediately delivered and there will
+# be no tracking overhead.
+delayedDeliveryEnabled=true
+
+# Control the tick time for when retrying on delayed delivery,
+# affecting the accuracy of the delivery time compared to the scheduled time.
+# Default is 1 second.
+delayedDeliveryTickTimeMillis=1000
+```
+
+### producer 
+Here's an example for Delayed message producer
+```java
+// message to be delivered at the configured delay interval
+producer.newMessage().deliverAfter(3L, TimeUnit.Minute).value("Hello 
Pulsar!").send();
+```
 
 Review comment:
   @pouledodue there is  no implementation on python client.


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-09-30 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r329529414
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memoey after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
+The diagram below illustrates concept:
+
+![Delayed Message Delivery](assets/message_delay.png)
+
+broker save message without any check. when consumer consume message, if 
message is delayed type, message will add to DelayedDeliveryTracker. 
subscription will check and get timeout messages from DelayedDeliveryTracker.
+
+### broker 
+Delayed Message Delivery is enabled by default on broker conf as below
+```
+# Whether to enable the delayed delivery for messages.
+# If disabled, messages will be immediately delivered and there will
+# be no tracking overhead.
+delayedDeliveryEnabled=true
+
+# Control the tick time for when retrying on delayed delivery,
+# affecting the accuracy of the delivery time compared to the scheduled time.
+# Default is 1 second.
+delayedDeliveryTickTimeMillis=1000
+```
+
+### producer 
+Here's an example for Delayed message producer
+```java
+// message to be delivered at the configured delay interval
+producer.newMessage().deliverAfter(3L, TimeUnit.Minute).value("Hello 
Pulsar!").send();
+```
 
 Review comment:
there is  no implementation on python client.


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


With regards,
Apache Git Services


[GitHub] [pulsar] xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-09-30 Thread GitBox
xujianhai666 commented on a change in pull request #5293: [ISSUE #4675][docs] 
Add *Delayed Message Delivery* 
URL: https://github.com/apache/pulsar/pull/5293#discussion_r329526225
 
 

 ##
 File path: site2/docs/concepts-messaging.md
 ##
 @@ -411,4 +411,32 @@ Message deduplication makes Pulsar an ideal messaging 
system to be used in conju
 
 > More in-depth information can be found in [this 
 > post](https://streaml.io/blog/pulsar-effectively-once/) on the [Streamlio 
 > blog](https://streaml.io/blog)
 
+## Delayed Message Delivery
+Delayed Message Delivery enables you to consume new message later, instead of 
immediately as normal. In this mechanism, messages is stored in memoey after 
publish to broker, after specific delayed time passed, the message will be 
delivered to consumer.  
 
 Review comment:
   @pouledodue done


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


With regards,
Apache Git Services