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

ref_collection_lidvid field in some node registries populated using older versions of registry loader tools create string fields in the schema vs. the expected list types #159

Closed
jordanpadams opened this issue Jan 31, 2023 · 1 comment
Assignees
Labels
B13.1 bug Something isn't working i&t.done s.medium Medium level severity

Comments

@jordanpadams
Copy link
Member

jordanpadams commented Jan 31, 2023

Checked for duplicates

Yes - I've already checked

πŸ› Describe the bug

I believe GEO has some of these metadata fields, as an example for updating this.

πŸ•΅οΈ Expected behavior

All fields are lists

πŸ“œ To Reproduce

  1. Query a registry that contains String fields instead of list fields
  2. Node that they are String fields and should be lists :-)

πŸ–₯ Environment Info

N/A

πŸ“š Version of Software Used

N/A

🩺 Test Data / Additional context

No response

πŸ¦„ Related requirements

NASA-PDS/registry-common#29

βš™οΈ Engineering Details

N/A

@jordanpadams jordanpadams added bug Something isn't working needs:triage labels Jan 31, 2023
@jordanpadams jordanpadams self-assigned this Jan 31, 2023
@jordanpadams jordanpadams assigned jimmie and unassigned jordanpadams Jan 31, 2023
@jordanpadams jordanpadams changed the title Fields in node registries using old versions of registry loader tools load string fields, not the expected list types ref_collection_lidvid field in some node registries populated using older versions of registry loader tools create string fields in the schema vs. the expected list types Feb 22, 2023
@jimmie
Copy link
Member

jimmie commented Feb 27, 2023

This would require a script to determine which documents have atomic values and convert them to lists. Since the API (and I assume ingestion tools) have been modified to tolerate heterogeneous types we are going to punt on this for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B13.1 bug Something isn't working i&t.done s.medium Medium level severity
Projects
None yet
Development

No branches or pull requests

3 participants