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.

Sign up for GitHub

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

Jump to bottom

Add ldms_schema_name_get API #1220

Merged
merged 2 commits into from
Jul 3, 2023
Merged

Add ldms_schema_name_get API #1220

merged 2 commits into from
Jul 3, 2023

Conversation

tom95858
Copy link
Collaborator

@tom95858 tom95858 commented Jun 23, 2023 •
edited
Loading

  • Add an API that returns an LDMS schema name given the ldms_schema_t structure.
  • Remove unnecessary x-digit code that is part of libc in ctype.h
  • Fix bug in LDMS_V_RECORD_ARRAY logic that miscalculates offset of next metric
  • Tweak record and record-array documentation

@tom95858
Add a ldms_schema_name_get service
7d732e5 
This service can be used by plugins passed arbitrary sets to
determine the schema name associated with the set.

- Logic in ldms.c that duplicated standard lib-c ctype
logic was removed.

- Some LDMS record data types were renamed to be consistent
with other data types in core.

- Various documentation improvements and spelling errors were
made.

- Fix bug in LDMS_V_RECORD_ARRAY logic that miscalculates offset of next
metric
@tom95858 tom95858 merged commit 636a1c6 into OVIS-4 Jul 3, 2023
14 checks passed
@tom95858 tom95858 deleted the ldms-schema-name branch July 3, 2023 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tom95858