Skip to content

hasRecipe should consult recipe cache#124

Open
cha0s wants to merge 3 commits intoQelifern:1.19.2from
cha0s:cache-recipe-lookups
Open

hasRecipe should consult recipe cache#124
cha0s wants to merge 3 commits intoQelifern:1.19.2from
cha0s:cache-recipe-lookups

Conversation

@cha0s
Copy link

@cha0s cha0s commented Sep 1, 2022

No description provided.

@cha0s
Copy link
Author

cha0s commented Sep 1, 2022

If desired, I can provide a full server snapshot of a fledgling ATM7 TTS server that demonstrates the problem (and the fix if applied).

@cha0s
Copy link
Author

cha0s commented Sep 26, 2022

Server-destroying performance issues at least deserve a review, when someone else has written all the code for you. :)

@everettperiman
Copy link

everettperiman commented Aug 2, 2023

Just wanted to bring this up. I pulled the current 1.20 branch as I was having severe server performance impacts associated with this mod on ATM9. After implementing this fix I had stopped having performance issues with this mod.

@cha0s
Copy link
Author

cha0s commented Dec 1, 2023

Bump

@fubz
Copy link

fubz commented Mar 3, 2024

@Qelifern This needs to be pushed ASAP. IronFurnaces performance is awful due to lack of cache usage. One furnace takes the server from 20 TPS to 3 TPS

@fubz
Copy link

fubz commented Mar 3, 2024

Looks we we have the change here :) Thanks!

b36d09b#diff-b090dc51498ceb387143bdf6450b97cbbf4f157a34da9640f00ad6964b67efe5R198

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.

4 participants