Particular / NServiceBus

Build, version, and monitor better microservices with the most powerful service platform for .NET
https://particular.net/nservicebus/
Other
2.1k stars 647 forks source link

Limit message size on the learning transport to Min(SupportedTransports) #4664

Closed andreasohlund closed 7 years ago

andreasohlund commented 7 years ago

At the time of writing this would be 64kb (ASQ).

We would throw telling users that some transports won't handle this and suggest the Databus/Claim check pattern.

We would also provide a config option to up the limit if needed.

cc @Particular/azure-maintainers

@Particular/rabbitmq-transport-maintainers what's the limit on rabbit?

Scooletz commented 7 years ago

👍

danielmarbach commented 7 years ago

Like it

bording commented 7 years ago

RabbitMQ does not have a maximum message size that is enforced, but a really large message would certainly slow things down inside the broker.

OK, technically there is a max length, because the total payload size is included in the AMQP message header frame as an unsigned 64-bit value! 😄

SeanFeldman commented 7 years ago

Just the fact that out of the box transport will raise the awareness about transports potentially having a message size limit is a move in the right direction. 💯%

yvesgoeleven commented 7 years ago

👍