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

Occasional ConcurrentModificationException when loading datapacks #3969

Closed
xllifi opened this issue Jul 25, 2024 · 5 comments · Fixed by #3970
Closed

Occasional ConcurrentModificationException when loading datapacks #3969

xllifi opened this issue Jul 25, 2024 · 5 comments · Fixed by #3970
Labels
bug Something isn't working priority:high High priority PRs that need review and work now. Review these first. registry-sync Pull requests and issues related to registry sync

Comments

@xllifi
Copy link

xllifi commented Jul 25, 2024

Description:
Only happens when Fabric API is present. 50 launches without Fabric API mod were all successful.
This doesn't occur on every load. That's why I've written a node.js script that helped me debug this behavior: https://hst.sh/siloravisa.js

Relevant info:
Dungeons and Taverns: https://modrinth.com/datapack/dungeons-and-taverns
Log: https://hst.sh/aluripatij.log

Using:
Java Temurin 21.0.2.13
Loader 0.15.11
Minecraft 1.21
Fabric API 0.100.7+1.21
Dungeons and Taverns 4.3

Reproduction:

  1. Add Fabric API to mods, add Dungeons and Taverns as either mod or datapack
  2. Run the server multiple times. Either using a script or manually
@apple502j apple502j added bug Something isn't working registry-sync Pull requests and issues related to registry sync priority:high High priority PRs that need review and work now. Review these first. labels Jul 25, 2024
@davepusey
Copy link

Been seeing exactly the same problem on a small community server I admin.
We do a daily auto-restart, and every ~10 days it takes two attempts because the first attempt fails with this error.

We don't even have "Dungeons and Taverns", just a couple of basic mods, plus a bunch of the Vanilla Tweaks datapacks.

Log file from last night...

2024-07-28-3.log

@xllifi
Copy link
Author

xllifi commented Jul 29, 2024

Been seeing exactly the same problem on a small community server I admin. We do a daily auto-restart, and every ~10 days it takes two attempts because the first attempt fails with this error.

For now you can build Fabric API from apple502j's fork and use it for your server. I did this and tested with 50 launches, not a single crash.

Here's my build for 1.21 if you don't want to do it yourself (remove .zip and use the file as a replacement for Fabric API):
fabric-api-0.100.7+local-unknown.jar.zip
You should totally build it yourself though.

@modmuss50
Copy link
Member

Seems like a few people are having this crash suddenly, ill merge and release the fix shortly. I was going to wait until the next feature release, but it seems more important. Please do make sure to report issues as you as you see them, they cannot be fixed if we dont know about them.

@xllifi xllifi changed the title Occasional ConcurrentModificationException when using Dungeons and Taverns (Minecraft 1.21) Occasional ConcurrentModificationException when loading datapacks Jul 29, 2024
@xllifi
Copy link
Author

xllifi commented Jul 29, 2024

Seems like a few people are having this crash suddenly,

I don't think that's the case, more probable is that this issue is now searchable (at least on Fabric discord), so people don't have to go through all of their mods just to find out that the last suspect, Fabric API, is guilty.

@davepusey
Copy link

To be honest, I was still investigating it is a datapack issue, not a Fabric/mod issue. It was only when looking for something else that I came across this report and had the a-ha moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority:high High priority PRs that need review and work now. Review these first. registry-sync Pull requests and issues related to registry sync
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants