View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005379 | 10000-014: PubSub | Spec | public | 2020-01-08 16:29 | 2020-06-17 13:39 |
| Reporter | Zbynek Zahradnik | Assigned To | Matthias Damm | ||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | no change required | ||
| Summary | 0005379: 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. | ||||
| Tags | No tags attached. | ||||
| Commit Version | |||||
| Fix Due Date | |||||
|
|
There are different combinations of configuraiton options that make no sense or are invalid. If a configuraiton is invalid or cannot be used by the application within its context, the corresponding PubSub objects are set to Error State. |
|
|
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. |
|
|
After the discussion in the meeting today we agreed that this is an invalid configuration. |
|
|
Agreed to no-fix in virtual F2F. |
| 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 |