-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Maximum Message Size #3
Comments
I believe that the (only?) valid argument is for urgent data (such as CAP warnings). For "normal" earth system data, the default method, get the message, fetch the data through URL is good enough for our operations. |
This issue seems to be a duplicate of #6, but let me repeate my last comment from that issue:
|
|
Truncation is not going to work, especially for CAP, the main format of concern. It is XML and important information could be scatterred anywhere in the content. it is not organized to present the most important information first. The purpose of the messages is to let people decide whether they want the full message.
That's all. The essential information needed needs to be in the message itself. The format includes geometry and "start_datetime" and "end_datetime", and the type of event should fall naturally out of the topic hierarchy. If the subtopic and geometry information is combined, that is probably good enough, and including any of the content is redundant (I mean... if the whole warning is < 4kb, then it is fine to include it, but if truncation enters into the story, then do not include any content, since it is just a guess whether the first 4Kb are useful/relevent/crucial.) sample Canadian warnings: https://hpfx.collab.science.gc.ca/20230113/WXO-DD/alerts/cap/20230113/CWUL/10/
The first 4K of a 310K CAP is unlikely to be helpful. It will contain information on a subset of regions affected by the alert... the rest of the regions won't be included. |
I agree with @petersilva, neither Global Broker nor any other system shall truncate the notifications. The CAP messages can be compressed to less than 20% of their size which is sometimes more than the magic 4kB but I would not be too worried about it now, these messages (or notifications about such messages) are not so frequent. |
There is no truncation envisaged. |
Shall we close the issue with the note: agreed that first 4kB will be set as maximum (with adjustment if necessary due to more experience), during pilot phase with added warning comment from GB - later GB will discard greater messages? |
TT-WISMD 2023-09-11:
Relevant doc updates:
|
PR in #63 |
Note: this will NOT be updated in the Guide since we already have a provision that covers:
|
None of the MQP message protocols define a maximum message size, but message queuing brokers are optimized for small messages. Brokers of all MQP achieve their highest performance forwarding many small messages, rather than few large ones. Even rare Large messages can have an outsized effect on performance by requiring buffers on brokers to expand for those rare cases, so keeping messages are uniformly "small" is necessary for best performance.
The renewal of MQP protocol usage in GTS aims to have only announcements of products, not to transport the data itself, which is a particularly easy way to get smaller messages. However, many people with long experience with other messaging protocols expect the old methods, where products themselves were included in the files ( examples #36 ) and the Aviation people seem to want data to be included in the body as well. This is unlikely to scale well.
The aviation people have mentioned 8MB as a maximum product size for aviation. We believe that, whatever method is adopted by other groups, they will eventually find themselves wanting to send products larger than the maximum, and will eventually be faced with two choices: either they add support for links to data, as we have already done, or they introduce segmentation. Having that choice happen at a large message size is likely worse for performance.
The old GTS 386 limit was 500,000 bytes, which already sounds relatively large. That's already a high number in my view, but I think the committee was interested in doing some performance studies before settling on a recommendation. So this is a placeholder for now.
(transferred from wmo-im/GTStoWIS2#44 )
The text was updated successfully, but these errors were encountered: