-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Make CSV upload limit configurable #8228
Conversation
Can you add this configuration value to the docs? |
Kibana's Upload CSV feature isn't intended for gigantic import jobs, so I originally set a sane default of 1GB. Some users exprssed a desire to import slightly larger files, they should be able to import something that's 1.1GB without being blocked by an arbitrary limit. So I've made the limit configurable via kibana.yml. This change includes a few pieces: * Added optional `kibana.addDataMaxBytes` key to kibana.yml with 1GB default * Set upload data route payload limit based on new config value * Updated help text in UI to use the dynamic config value on the parse csv page * Updated parse csv page to check file size and fail early if the selected file is too big Resolves: elastic#7671
@epixa done |
LGTM |
1 similar comment
LGTM |
@@ -21,6 +22,8 @@ modules.get('apps/management') | |||
const maxSampleRows = 10; | |||
const maxSampleColumns = 20; | |||
|
|||
this.maxBytesFormatted = numeral(addDataMaxBytes).format('0 b'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't seem this is used anywhere. Did you mean to use it in the formatted error message on line 63?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's used in the view
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the speedy review guys! |
@Bargs Please make sure this is backported to |
--------- **Commit 1:** Make CSV upload limit configurable Kibana's Upload CSV feature isn't intended for gigantic import jobs, so I originally set a sane default of 1GB. Some users exprssed a desire to import slightly larger files, they should be able to import something that's 1.1GB without being blocked by an arbitrary limit. So I've made the limit configurable via kibana.yml. This change includes a few pieces: * Added optional `kibana.addDataMaxBytes` key to kibana.yml with 1GB default * Set upload data route payload limit based on new config value * Updated help text in UI to use the dynamic config value on the parse csv page * Updated parse csv page to check file size and fail early if the selected file is too big Resolves: #7671 * Original sha: 0503aa8 * Authored by Matthew Bargar <mbargar@gmail.com> on 2016-09-12T18:13:44Z
--------- **Commit 1:** Make CSV upload limit configurable Kibana's Upload CSV feature isn't intended for gigantic import jobs, so I originally set a sane default of 1GB. Some users exprssed a desire to import slightly larger files, they should be able to import something that's 1.1GB without being blocked by an arbitrary limit. So I've made the limit configurable via kibana.yml. This change includes a few pieces: * Added optional `kibana.addDataMaxBytes` key to kibana.yml with 1GB default * Set upload data route payload limit based on new config value * Updated help text in UI to use the dynamic config value on the parse csv page * Updated parse csv page to check file size and fail early if the selected file is too big Resolves: #7671 * Original sha: 0503aa8 * Authored by Matthew Bargar <mbargar@gmail.com> on 2016-09-12T18:13:44Z
[backport] PR #8228 to 5.x - Make CSV upload limit configurable
[backport] PR #8228 to 5.0 - Make CSV upload limit configurable
--------- **Commit 1:** Make CSV upload limit configurable Kibana's Upload CSV feature isn't intended for gigantic import jobs, so I originally set a sane default of 1GB. Some users exprssed a desire to import slightly larger files, they should be able to import something that's 1.1GB without being blocked by an arbitrary limit. So I've made the limit configurable via kibana.yml. This change includes a few pieces: * Added optional `kibana.addDataMaxBytes` key to kibana.yml with 1GB default * Set upload data route payload limit based on new config value * Updated help text in UI to use the dynamic config value on the parse csv page * Updated parse csv page to check file size and fail early if the selected file is too big Resolves: elastic#7671 * Original sha: bdc2f855000d3d153e85c16deef81a17f853d69c [formerly 0503aa8] * Authored by Matthew Bargar <mbargar@gmail.com> on 2016-09-12T18:13:44Z Former-commit-id: 7479654
[backport] PR elastic#8228 to 5.x - Make CSV upload limit configurable Former-commit-id: c417cac
Kibana's Upload CSV feature isn't intended for gigantic import jobs, so
I originally set a sane default of 1GB. Some users exprssed a desire to
import slightly larger files, they should be able to import something
that's 1.1GB without being blocked by an arbitrary limit. So I've made
the limit configurable via kibana.yml.
This change includes a few pieces:
kibana.addDataMaxBytes
key to kibana.yml with 1GBdefault
csv page
selected file is too big
Resolves: #7671