Skip to content
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

TS 26.532: Use of "NULL" in DataAggregationFunctionType may cause issues. #117

Open
davidjwbbc opened this issue Feb 27, 2024 · 3 comments
Assignees
Labels
3GPP Rel-17 Issues relating to 3GPP Release 17 specifications. 3GPP Rel-18 Issues relating to 3GPP Release 18 specifications. 3GPP TS 26.532 Issues Relating to SA4's: "Data Collection and Reporting; Protocols and Formats" specification. 5G Core: Data Collection Adopted Clarification Improvement

Comments

@davidjwbbc
Copy link

davidjwbbc commented Feb 27, 2024

Context

Table 6.3.3.2-1 in TS 26.532 V17.2.0 and V18.1.0 defines the DataAggregationFunctionType enumeration. This enumeration contains a value of "NULL".

Problem

The term NULL is used in lots of different computer languages to mean that a value is not defined. However in the case of this enumeration the value is defined, it is just that the meaning is that there is no aggregation function. This can lead to confusion between the absence of the value and the value being defined but indicating no change to the data. It can also clash with parsers trying to interpret the value NULL specially and implementations where NULL also means not defined.

Suggested Solution

The enumeration should use a term other than NULL to avoid confusion. A better term could be NONE (there is no aggregation function) or IDENTITY (the aggregation function does not change the data).

@davidjwbbc davidjwbbc added Improvement Clarification 3GPP Rel-18 Issues relating to 3GPP Release 18 specifications. 3GPP TS 26.532 Issues Relating to SA4's: "Data Collection and Reporting; Protocols and Formats" specification. 5G Core: Data Collection labels Feb 27, 2024
@rjb1000 rjb1000 added the 3GPP Rel-17 Issues relating to 3GPP Release 17 specifications. label Feb 27, 2024
@rjb1000
Copy link
Contributor

rjb1000 commented Feb 27, 2024

Given that this issue affects the provisioning of event exposure in the standalone Data Collection AF (i.e. for exposure of the basic SA2 events to NWDAF), I think changes to both Rel-17 and Rel-18 are justified, even though 5G-MAG Reference Tools is only implementing Rel-18.

The Rel-17 fix could potentially be folded in to TS 26.532 CR0007 that is already addressing #114.

@rjb1000 rjb1000 added the Adopted label Mar 7, 2024
@rjb1000
Copy link
Contributor

rjb1000 commented Apr 2, 2024

Change Requests contributed to SA4#127-bis-e for agreement:

  • TS 26.532
    • Rel-17 CR0007r2 "[EVEX] Essential maintenance" in S4-240553.
    • Rel-18 CR0009 "[EVEX, TEI18] Essential maintenance" in S4-240554.

@rjb1000
Copy link
Contributor

rjb1000 commented Apr 23, 2024

Change Requests agreed without revision at SA4#127-bis-e:

  • TS 26.532
    • Rel-17 CR0007r2 "[EVEX] Essential maintenance" in S4-240553.
    • Rel-18 CR0009 "[EVEX, TEI18] Essential maintenance" in S4-240554.

These will go to SA#104 for approval in June.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3GPP Rel-17 Issues relating to 3GPP Release 17 specifications. 3GPP Rel-18 Issues relating to 3GPP Release 18 specifications. 3GPP TS 26.532 Issues Relating to SA4's: "Data Collection and Reporting; Protocols and Formats" specification. 5G Core: Data Collection Adopted Clarification Improvement
Projects
Status: Agreed/endorsed by Working Group
Development

No branches or pull requests

2 participants