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.

Sign up for GitHub

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

Jump to bottom

[ML] FileViz - inaccurate file size handling #25114

Closed
pheyos opened this issue Nov 5, 2018 · 4 comments
Closed

[ML] FileViz - inaccurate file size handling #25114

pheyos opened this issue Nov 5, 2018 · 4 comments
Assignees
@peteharverson
Labels
bug Fixes for quality problems that affect the customer experience :ml v6.5.1 v6.6.0 v7.0.0

Comments

@pheyos
Copy link
Member

pheyos commented Nov 5, 2018

Found in version

  • 6.5.0-bc1

Browser

  • Firefox

Steps to reproduce

  • Navigate to the ML Data Visualizer / import data
  • Select a file of size 104857600 bytes (= 100MB) for upload

Expected result

  • The file upload works without errors

Actual result

  • An error message is displayed:
    fileviz_file_size_error

Additional information

  • The error messages should format the file size numbers as bytes.
  • When uploading a 104857599 bytes file, it is accepted at first, but then the following error is displayed:
    fileviz_file_upload_size_error
@pheyos pheyos added bug Fixes for quality problems that affect the customer experience :ml labels Nov 5, 2018
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui

@peteharverson peteharverson self-assigned this Nov 6, 2018
@pheyos
Copy link
Member Author

pheyos commented Nov 7, 2018

On 6.5.0-bc4 the Payload content length error (mentioned above under Additional information) didn't occur any more.

@peteharverson peteharverson mentioned this issue Nov 7, 2018
Merged
1 task
@peteharverson
Copy link
Contributor

Reopening issue pending backport to v6.5.1

@peteharverson peteharverson reopened this Nov 7, 2018
@spalger spalger added v6.5.1 and removed v6.5.1 labels Nov 14, 2018
@peteharverson
Copy link
Contributor

Closing after backporting #25295 to 6.5.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@peteharverson peteharverson

Labels
bug Fixes for quality problems that affect the customer experience :ml v6.5.1 v6.6.0 v7.0.0
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@spalger @pheyos @peteharverson @elasticmachine