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

Update staging and production Registry APIs to increase window limit #291

Closed
jordanpadams opened this issue Mar 16, 2023 · 8 comments
Closed

Comments

@jordanpadams
Copy link
Member

💡 Description

Per NASA-PDS/registry-mgr#53 and NASA-PDS/registry-mgr#54, we updated registry-mgr to increase the OpenSearch window limit to 1000000000. This will help resolve issues in pagination and the number of hits being displayed (NASA-PDS/pds-api#199)

@jordanpadams jordanpadams changed the title Update Registry API config to increase window limit Update staging and production Registry APIs to increase window limit Mar 16, 2023
@jordanpadams
Copy link
Member Author

@alexdunnjpl @jimmie can we coordinate this in the next week to see if it is works? once we set it, do we need to do anything else for it to take affect?

@alexdunnjpl
Copy link
Contributor

@jordanpadams sure, though I'm not sure what I'm bringing to the table on this one

@alexdunnjpl
Copy link
Contributor

@jimmie @jordanpadams any plans for this ticket?

@jordanpadams
Copy link
Member Author

@alexdunnjpl this needs to be done ASAP. @jimmie how do we do this? Do we just need to run some command to update the OpenSearch config or through Kibana?

@jordanpadams
Copy link
Member Author

also is this part of the Registry Terraform scripts?

@jimmie
Copy link
Contributor

jimmie commented Apr 11, 2023

The result window is specified at the time the index is created - so it should be added to registry-mgr. When I did the provenance re-indexing, all max_result_window values were set to 1,000,000,000. It may have been present prior but it is certainly there now. This can be checked by examining the index: https://<opensearch_url>/registry

@jordanpadams
Copy link
Member Author

thanks @jimmie

@jordanpadams
Copy link
Member Author

just verified on pds-en registry:

..
        "provided_name" : "registry",
        "max_result_window" : "1000000000",
        "creation_date" : "1674616050693",
        "analysis" : {
          "normalizer" : {
            "keyword_lowercase" : {
              "filter" : [ "lowercase" ],
              "type" : "custom"
            }
          }
        },

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants