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

artpar's attempt #11

Closed
wants to merge 18 commits into from
Closed

artpar's attempt #11

wants to merge 18 commits into from

Conversation

artpar
Copy link
Contributor

@artpar artpar commented Jan 2, 2024

Trying to do multiple things at once

  • memory mapped files
  • parallel processing
  • aggressive caching for object creations/parse paths

@artpar artpar changed the title Artpar's attempt artpar's attempt Jan 2, 2024
@gunnarmorling
Copy link
Owner

Hey @artpar, thanks a lot! Could your rebase this one to latest main and squash everything into a single commit? Right now, I am getting an NPE when running it:

java.lang.NullPointerException: Cannot invoke "java.lang.Double.doubleValue()" because "doubleValue" is null
	at dev.morling.onebrc.CalculateAverage_artpar$ReaderRunnable.run(CalculateAverage_artpar.java:209)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:317)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642)
	at java.base/java.lang.Thread.run(Thread.java:1583)

@artpar artpar closed this by deleting the head repository Jan 3, 2024
@artpar
Copy link
Contributor Author

artpar commented Jan 3, 2024

sorry i messed up my git. recreated from scratch
#27

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.

6 participants