View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0010174CTT UA Scripts6 - Performance Problempublic2025-02-28 14:472025-06-03 05:22
ReporterYannik Klaass Assigned ToYannik Klaass  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Fixed in Version1.04.509 
Summary0010174: Performance of CollectNetworkMessageData() function can be improved
Description

The CollectNetworkMessageData() function currently processes data of which the result is not needed and can be skipped:

  • When decoding a message with the purpose of only getting the PublisherId, the decoding of the rest of the message can be skipped after the PublisherId
  • When the number of new messages in the message cache exceeds the 'MaxNumberOfMessages', processing the exceeding messages should be skipped, as they get sliced from the result

Furthermore the decoding of UADP PubSub messages involves inverting serveral ByteStrings which could be done faster when using a binary-side invert() method (see Mantis 0010171).

TagsNo tags attached.
Files Affected

/library/PubSub/PubSubUtilities.js
/library/PubSub/UADPMessageDissector.js

Activities

Yannik Klaass

2025-04-07 14:37

developer   ~0022601

  • Adding option to return right after the PublisherId has been decoded in UaPubSubTraceNetworkMessage.fromRawMessage() and CollectNetworkMessageData()
  • Skipping processing of remaining messages in CollectNetworkMessageData(), if MaxNumberOfMessages is exceeded
  • Changing invertByteString() function to use the invert() function of the CttByteString class

Paul Hunkar

2025-06-03 05:22

administrator   ~0022905

Multiple people reviewed code , all agree to changes, closed issue

Issue History

Date Modified Username Field Change
2025-02-28 14:47 Yannik Klaass New Issue
2025-03-01 07:09 Paul Hunkar Assigned To => Yannik Klaass
2025-03-01 07:09 Paul Hunkar Status new => assigned
2025-03-31 08:57 Yannik Klaass Files Affected => /library/PubSub/PubSubUtilities.js
/library/PubSub/UADPMessageDissector.js
2025-04-07 14:37 Yannik Klaass Status assigned => resolved
2025-04-07 14:37 Yannik Klaass Resolution open => fixed
2025-04-07 14:37 Yannik Klaass Note Added: 0022601
2025-06-03 05:22 Paul Hunkar Status resolved => closed
2025-06-03 05:22 Paul Hunkar Fixed in Version => 1.04.509
2025-06-03 05:22 Paul Hunkar Note Added: 0022905