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

Implementation using memory mapped file #10

Closed
wants to merge 1 commit into from

Conversation

bjhara
Copy link
Contributor

@bjhara bjhara commented Jan 2, 2024

Implementation entry using memory mapped file (CalculateAverage_bjhara.java)

On my system (Ryzen 7 5700G, 8 cores, 32 GB RAM) it takes appoximately 17 seconds (the reference implementation takes ~2m17s).

@royvanrijn
Copy link
Contributor

royvanrijn commented Jan 2, 2024

This is awesome, learned about memory mapped files from looking at your solution. I've learned from your approach and shoehorned it into my best version, which made it even faster. On my MacBook Pro (M2) it now runs in <5 seconds 🚀

@gunnarmorling
Copy link
Owner

I've added the missing license header to the source file and merged. Congrats, you're leading now with 00:38:510 on the official evaluation environment!

@twobiers
Copy link
Contributor

twobiers commented Jan 2, 2024

Cool, learned about a new API. Will try to integrate it in my branch to get hands on experience. Kudos to you!

@javamann
Copy link

javamann commented Jan 3, 2024

Really cool approach! Your knowledge of Streams beats the crap out of mine. I really like your splitFileChannel. There is a lot of good concepts that I'm going to steal for a side project of mine.

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

Successfully merging this pull request may close these issues.

5 participants