View Issue Details

IDProjectCategoryView StatusLast Update
000698610000-003: Address SpaceSpecpublic2021-09-28 16:44
ReporterDavid Levine Assigned ToJeff Harding  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product Version1.05.00 RC1 
Fixed in Version1.05.00 
Summary0006986: Description of arrayDimensions in StructureField incorrect, and wording can be improved of paragraph
Description

2nd sentence says "The number of elements shall be equal to the value of the valueRank field."
This should say "The number of dimensions shall be equal to the value of the valueRank field."

The paragraph would be more readable and make more sense by making it this:

"The number of dimensions shall be equal to the value of the valueRank field.
This field shall be null if valueRank ≤ 0, otherwise the value in each dimension specifies the maximum supported length of each dimension. If the maximum length of a dimension is unknown the value shall be 0.
Regardless of the number of dimensions, the maximum number of elements in an array transferred on the wire is 2147483647 (max Int32).
"

This would help eliminate confusion over the meaning of a non-zero value in a dimension. It should not be interpreted as a constant length, it is meant as a maximum length.

Question: are ValueRanks of 0, -2, and -3 valid? What is the use case for this?

If these are allowed then fields with this ValueRank will be extremely difficult for a client to handle all the edge cases, especially if it is allowed to change for each value.
IMO it should explicitly define a ValueRank of 0, -2 and -3 as illegal.

TagsNo tags attached.
Commit Version
Fix Due Date

Activities

David Levine

2021-06-04 11:03

developer   ~0014468

Revise last line in description to
"The maximum number of elements that can be transferred on the wire is 2147483647 (max Int32)."

This should help eliminate confusion over whether this restriction applies to a single dimension vs all dimensions.
If does not help anyone figure out how to transfer more than this number of elements, but that's a different issue - it implies a natural limit on the max length of an array for servers that do not support indexing.

David Levine

2021-06-04 18:35

developer   ~0014469

Spoke with Paul, and I know that all ValueRanks are supported, so ignore the question about ValueRank <= 0. The other comments are still valid.

Jeff Harding

2021-09-27 17:44

developer   ~0015028

Clarified the definition in line with the suggested text proposed.

Jim Luth

2021-09-28 16:44

administrator   ~0015056

Agreed to changes edited in 1.05.8 RC1 post edits.

Issue History

Date Modified Username Field Change
2021-06-04 10:54 David Levine New Issue
2021-06-04 11:03 David Levine Note Added: 0014468
2021-06-04 18:35 David Levine Note Added: 0014469
2021-09-07 16:08 Jim Luth Assigned To => Jeff Harding
2021-09-07 16:08 Jim Luth Status new => assigned
2021-09-27 17:44 Jeff Harding Status assigned => resolved
2021-09-27 17:44 Jeff Harding Resolution open => fixed
2021-09-27 17:44 Jeff Harding Fixed in Version => 1.05.00
2021-09-27 17:44 Jeff Harding Note Added: 0015028
2021-09-28 16:44 Jim Luth Status resolved => closed
2021-09-28 16:44 Jim Luth Note Added: 0015056