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

hasRecipe should consult recipe cache #124

Open
wants to merge 3 commits into
base: 1.19.2
Choose a base branch
from

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