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 documents size limit #141

Conversation

junqiu-lei
Copy link
Member

@junqiu-lei junqiu-lei commented Dec 22, 2022

Signed-off-by: Junqiu Lei junqiu@amazon.com

Description

Since we now have time filters to limit query by time, update default query document size limit to 1000, the max is from index.max_result_window size, reference here.

Also in this PR include: disable update button when documents size invalid and fix error message on refreshDataLayerRender on topNav filter bar

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Junqiu Lei <junqiu@amazon.com>
@junqiu-lei junqiu-lei added the enhancement New feature or request label Dec 22, 2022
@junqiu-lei junqiu-lei requested a review from a team December 22, 2022 19:50
@junqiu-lei junqiu-lei self-assigned this Dec 22, 2022
@junqiu-lei junqiu-lei changed the title Add delete layer warning modal Update documents size limit Dec 22, 2022
Signed-off-by: Junqiu Lei <junqiu@amazon.com>
@junqiu-lei junqiu-lei merged commit 66442fe into opensearch-project:feature/new-maps Dec 23, 2022
@junqiu-lei junqiu-lei deleted the update_documents_limit branch February 23, 2023 23:42
@cbeaujoin-stellar
Copy link

It seems limit is set to 10k not to index.max_result_window

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants