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

[Reporting/CSV] Add setting for max_concurrent_shard_requests in CSV export #161561

Closed
tsullivan opened this issue Jul 10, 2023 · 1 comment · Fixed by #170344
Closed

[Reporting/CSV] Add setting for max_concurrent_shard_requests in CSV export #161561

tsullivan opened this issue Jul 10, 2023 · 1 comment · Fixed by #170344
Assignees
Labels
enhancement New value added to drive a business result Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)

Comments

@tsullivan
Copy link
Member

The Elasticsearch team has recently added the max_concurrent_shard_requests parameter for opening point-in-time requests (i.e., /_pit) in Elasticsearch 8.8.2 or later. It defaults to 5, but some users might require a larger value.

We should expose this parameter kibana settings and use it in requests made by the CSV exporter, and allow administrators allowing them to adjust it if necessary.

@tsullivan tsullivan added enhancement New value added to drive a business result Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) labels Jul 10, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@tsullivan tsullivan changed the title [Reporting/CSV] Add kibana.yml setting to apply max_concurrent_shard_requests in CSV export [Reporting/CSV] Add setting for max_concurrent_shard_requests in CSV export Jul 10, 2023
rshen91 added a commit that referenced this issue Dec 5, 2023
## Summary

Closes #161561
This PR exposes the `max_concurrent_shards` setting in the schema for
customers for point in time CSV report generation.

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: Tim Sullivan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants