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

Extremely slow loading of big files #1724

Closed
mwu-tow opened this issue May 4, 2021 · 2 comments · Fixed by #1949
Closed

Extremely slow loading of big files #1724

mwu-tow opened this issue May 4, 2021 · 2 comments · Fixed by #1949
Assignees
Labels
p-medium Should be completed in the next few sprints

Comments

@mwu-tow
Copy link
Contributor

mwu-tow commented May 4, 2021

General Summary

Steps to Reproduce

I've created a 1GB file filled with random ASCII printable characters.
I've benchmarked the time needed to read it:

from Standard.Base import all
import Standard.Table

time_it label ~operation =
    start = System.nano_time
    result = operation
    end = System.nano_time
    time = end-start / 10^9
    IO.print_err (label + ": " + time.format "%.3fs")
    result

main =
    here.time_it "read" <|
        (Enso_Project.root / "tfile1G") . read

Expected Result

File loaded in, let's say, less than one second.
For comparison:

  • Rust program reads it in ~350ms
  • Python program reads it in ~850ms

Actual Result

File takes 140-160 seconds to read.

Similar results were observed with read_bytes, binary file and WSL environment.

Enso Version

Enso Compiler and Runtime
Version:    0.2.11
Built with: scala-2.13.5 for GraalVM 21.0.0.2
Built from: enso-0.2.11 @ 4ba8503575af07e4babe7cdcfd0430040180abb7
Running on: OpenJDK 64-Bit Server VM, GraalVM Community, JDK 11.0.10+8-jvmci-21.0-b06
            Windows 10 10.0 (amd64)
@iamrecursion iamrecursion added the p-medium Should be completed in the next few sprints label May 5, 2021
@iamrecursion
Copy link
Contributor

I assume you created the file with just effectively cat /dev/urandom > tfile1G?

@mwu-tow
Copy link
Contributor Author

mwu-tow commented May 5, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p-medium Should be completed in the next few sprints
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants