View Issue Details

IDProjectCategoryView StatusLast Update
000537910000-014: PubSubSpecpublic2020-06-17 13:39
ReporterZbynek Zahradnik Assigned ToMatthias Damm  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionno change required 
Summary0005379: In specific case(s), order of DataSetMessage-s in JSON NetworkMessage is relevant, and should be defined
Description

When NetworkMessageContentMask.DataSetMessageHeader == 0 and NetworkMessageContentMask.SingleDataSetMessage == 0, the JSON NetworkMessage will contain an array of payload fields for each DataSetMessage. The specification does not specify the order of these, and because the DataSetMessageHeader-s are not present, there is no way for the subscriber to identify which DataSetMessage is which.

The specification should define an order in this case - most likely, ascending DataSetWriterId-s.

TagsNo tags attached.
Commit Version
Fix Due Date

Activities

Matthias Damm

2020-06-09 20:22

developer   ~0012219

There are different combinations of configuraiton options that make no sense or are invalid.
Only header layouts define 'valid' or 'useful' combinations. Otherwise there is no dependency defined between configuration parameters.

If a configuraiton is invalid or cannot be used by the application within its context, the corresponding PubSub objects are set to Error State.

Zbynek Zahradnik

2020-06-10 06:47

developer   ~0012224

What MatthiasD writes in the note https://opcfoundation-onlineapplications.org/mantis/view.php?id=5379#c12219 is correct, but does not apply to this case.

The configuration I described does not fall into category "...configuraiton options that make no sense or are invalid.". It is quite meaningful, and only the missing order of datasets prevents it from being usable.

And conversely - I think we cannot ever make it usable by specifying header layout only, because order of the dataset messages is an information that belongs to the mapping part of the spec, and not to the header layout.

And, do we have any header layouts for JSON at all? I though the intent was to provide maximum flexibility for interop with other systems by allowing all kinds of possible combination, so this is precisely is the case which should be allowed, in case the other system we are communicating with wants this layout.

Matthias Damm

2020-06-17 13:38

developer   ~0012387

After the discussion in the meeting today we agreed that this is an invalid configuration.

Jim Luth

2020-06-17 13:39

administrator   ~0012388

Agreed to no-fix in virtual F2F.

Issue History

Date Modified Username Field Change
2020-01-08 16:29 Zbynek Zahradnik New Issue
2020-05-19 18:06 Jim Luth Assigned To => Matthias Damm
2020-05-19 18:06 Jim Luth Status new => assigned
2020-06-09 20:22 Matthias Damm Status assigned => resolved
2020-06-09 20:22 Matthias Damm Resolution open => not fixable
2020-06-09 20:22 Matthias Damm Note Added: 0012219
2020-06-10 06:47 Zbynek Zahradnik Status resolved => feedback
2020-06-10 06:47 Zbynek Zahradnik Resolution not fixable => reopened
2020-06-10 06:47 Zbynek Zahradnik Note Added: 0012224
2020-06-17 13:38 Matthias Damm Status feedback => resolved
2020-06-17 13:38 Matthias Damm Resolution reopened => no change required
2020-06-17 13:38 Matthias Damm Note Added: 0012387
2020-06-17 13:39 Jim Luth Status resolved => closed
2020-06-17 13:39 Jim Luth Note Added: 0012388