You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/03/07 14:45:14 UTC
[GitHub] [pulsar] codelipenghui opened a new issue #14581: Provide the delayed message publish rate stats
codelipenghui opened a new issue #14581:
URL: https://github.com/apache/pulsar/issues/14581
**Is your enhancement request related to a problem? Please describe.**
Currently, the producer stats contains the following information
```
{
"accessMode" : "Shared",
"msgRateIn" : 0.016669007231704083,
"msgThroughputIn" : 11.384931939253889,
"averageMsgSize" : 683.0,
"chunkedMessageRate" : 0.0,
"producerId" : 2,
"metadata" : { },
"address" : "/xxxx",
"connectedSince" : "2022-03-07T14:32:57.52849Z",
"producerName" : "xxx"
}
```
From the stats, we are not able to know whether the problem
is sending delayed messages or not, it will be useful for some cases
to troubleshoot problems.
**Describe the solution you'd like**
Add "delayedMsgRateIn" in the producer stats
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] shibd commented on issue #14581: Provide the delayed message publish rate stats
Posted by GitBox <gi...@apache.org>.
shibd commented on issue #14581:
URL: https://github.com/apache/pulsar/issues/14581#issuecomment-1062724218
@codelipenghui
The delays is placed in the `MessageMetadata`. This feature will make the broker additional a deserialization `MessageMetaData` when receiving the producer message. Can we accept the performance impact?
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Technoboy- commented on issue #14581: Provide the delayed message publish rate stats
Posted by GitBox <gi...@apache.org>.
Technoboy- commented on issue #14581:
URL: https://github.com/apache/pulsar/issues/14581#issuecomment-1063732930
> > @codelipenghui
> > The delays is placed in the MessageMetadata. This feature will make the broker additional a deserialization MessageMetaData when receiving the producer message. Can we accept the performance impact?
>
> Or, We need add new field `hasDeliver` to `CommandSend`, This can avoid deserialization the data of `MessageMetadata`. This is an API protocol change, We need to discuss first.
>
> @codelipenghui @Technoboy- PTAL
I will help to check first and then decide to send out a discussion email.
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] shibd commented on issue #14581: Provide the delayed message publish rate stats
Posted by GitBox <gi...@apache.org>.
shibd commented on issue #14581:
URL: https://github.com/apache/pulsar/issues/14581#issuecomment-1060783211
@codelipenghui I want to try it.
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Technoboy- commented on issue #14581: Provide the delayed message publish rate stats
Posted by GitBox <gi...@apache.org>.
Technoboy- commented on issue #14581:
URL: https://github.com/apache/pulsar/issues/14581#issuecomment-1063986821
> > > @codelipenghui
> > > The delays is placed in the MessageMetadata. This feature will make the broker additional a deserialization MessageMetaData when receiving the producer message. Can we accept the performance impact?
> >
> >
> > Or, We need add new field `hasDeliver` to `CommandSend`, This can avoid deserialization the data of `MessageMetadata`. This is an API protocol change, We need to discuss first.
> > @codelipenghui @Technoboy- PTAL
>
> I will help to check first and then decide to send out a discussion email.
Yes, you're right. If add this field in the `CommandSend`, it is duplicated with `MsgMetadata`, and deserialization is much heavier, so after discussion, we decide to close this issue now.
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Technoboy- closed issue #14581: Provide the delayed message publish rate stats
Posted by GitBox <gi...@apache.org>.
Technoboy- closed issue #14581:
URL: https://github.com/apache/pulsar/issues/14581
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] shibd commented on issue #14581: Provide the delayed message publish rate stats
Posted by GitBox <gi...@apache.org>.
shibd commented on issue #14581:
URL: https://github.com/apache/pulsar/issues/14581#issuecomment-1063641106
> @codelipenghui
The delays is placed in the MessageMetadata. This feature will make the broker additional a deserialization MessageMetaData when receiving the producer message. Can we accept the performance impact?
Or, We need add new field `hasDeliver` to `CommandSend`, This can avoid deserialization the data of `MessageMetadata`. This is an API protocol change, We need to discuss first.
@codelipenghui @Technoboy- PTAL
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org