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

Folia issue with #getLivingEntities #7221

Closed
Anon8281 opened this issue Jan 31, 2024 · 1 comment
Closed

Folia issue with #getLivingEntities #7221

Anon8281 opened this issue Jan 31, 2024 · 1 comment
Assignees
@Warriorrrr
Labels
duplicate
Milestone
0.100.2.0

Comments

@Anon8281
Copy link

What steps will reproduce the problem?

  1. Build Folia 1.20.4 from dev branch and install it
  2. Download latest Towny plugin
  3. Enjoy funny logs in terminal

What is the expected output?

Normal work

Towny version

0.100.1.7

Server version

Folia 1.20.4

Please use Pastebin.com to link the following files

[06:36:15 WARN]: [Towny] Global task for Towny v0.100.1.7 generated an exception
java.lang.IllegalStateException: Accessing entity state off owning region's thread
        at io.papermc.paper.util.TickThread.ensureTickThread(TickThread.java:79) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at org.bukkit.craftbukkit.v1_20_R3.entity.CraftPig.getHandle(CraftPig.java:67) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at org.bukkit.craftbukkit.v1_20_R3.entity.CraftPig.getHandle(CraftPig.java:8) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at org.bukkit.craftbukkit.v1_20_R3.entity.CraftEntity.isInWorld(CraftEntity.java:939) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at org.bukkit.craftbukkit.v1_20_R3.entity.CraftEntity.isValid(CraftEntity.java:393) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at org.bukkit.craftbukkit.v1_20_R3.CraftRegionAccessor.lambda$getLivingEntities$2(CraftRegionAccessor.java:321) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at java.lang.Iterable.forEach(Iterable.java:75) ~[?:?]
        at org.bukkit.craftbukkit.v1_20_R3.CraftRegionAccessor.getLivingEntities(CraftRegionAccessor.java:317) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at com.palmergames.bukkit.towny.tasks.MobRemovalTimerTask.run(MobRemovalTimerTask.java:96) ~[towny-0.100.1.7.jar:?]
        at com.palmergames.bukkit.towny.scheduling.impl.FoliaTaskScheduler.lambda$runRepeating$6(FoliaTaskScheduler.java:87) ~[towny-0.100.1.7.jar:?]
        at io.papermc.paper.threadedregions.scheduler.FoliaGlobalRegionScheduler$GlobalScheduledTask.run(FoliaGlobalRegionScheduler.java:179) ~[folia-1.20.4.jar:?]
        at io.papermc.paper.threadedregions.scheduler.FoliaGlobalRegionScheduler.tick(FoliaGlobalRegionScheduler.java:37) ~[folia-1.20.4.jar:?]
        at io.papermc.paper.threadedregions.RegionizedServer.globalTick(RegionizedServer.java:293) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at io.papermc.paper.threadedregions.RegionizedServer$GlobalTickTickHandle.tickRegion(RegionizedServer.java:148) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at io.papermc.paper.threadedregions.TickRegionScheduler$RegionScheduleHandle.runTick(TickRegionScheduler.java:404) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at ca.spottedleaf.concurrentutil.scheduler.SchedulerThreadPool$TickThreadRunner.run(SchedulerThreadPool.java:525) ~[folia-1.20.4.jar:git-Folia-"b0c8ebf"]
        at java.lang.Thread.run(Thread.java:833) ~[?:?]
@Anon8281 Anon8281 added the bug label Jan 31, 2024
@LlmDl
Copy link
Member

LlmDl commented Jan 31, 2024

Apparently it is a folia bug that started after 1.20.2. #7202

@LlmDl LlmDl closed this as not planned Won't fix, can't repro, duplicate, stale Jan 31, 2024
@LlmDl LlmDl added duplicate and removed bug labels Jan 31, 2024
@Warriorrrr Warriorrrr mentioned this issue Feb 18, 2024
Merged
1 task
@LlmDl LlmDl added this to the 0.100.2.0 milestone Feb 18, 2024
LlmDl added a commit that referenced this issue Feb 19, 2024
@LlmDl
- Remove magic values from spawnpoint ring offsets, courtesy of
c3bef16 
Warrior with PR #7267.
  - Fix mob removal on folia, courtesy of Warrior with PR #7262.
    - Closes #7261, #7221, #7202, #7253.
  - Make our custom registered commands identifiable, courtesy of
Warrior with PR #7263.
  - Move using_economy to the eco section of the config, courtesy of
Warrior with PR #7265.
  - Automatic config edit:
    - plugin.interfacing.using_economy moves to economy.using_economy
  - Remove uuid percentage tracking, courtesy of Warrior with PR #7266.
  - Make PlotClaim a runnable, courtesy of Warrior with PR #7268.
  - Reorganize test directory layout, courtesy of Warrior with PR #7269.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Warriorrrr Warriorrrr

Labels
duplicate
Projects
None yet
Milestone
0.100.2.0
Development

No branches or pull requests
3 participants
@LlmDl @Warriorrrr @Anon8281