-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Bundle spark #11093
Bundle spark #11093
Conversation
3ae21ed
to
7e3039e
Compare
This comment was marked as duplicate.
This comment was marked as duplicate.
it is deprecated and turned off by default now, but it shouldn't be removed entirely yet, since there are people which may still want to use it/need to use it |
disregard this comment, this is not gonna happen as this will turned off as default |
Should also add a config option to disable spark. |
f96b254
to
f05c2a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The We recommend installing the spark profiler as a replacement:
warning in the timings patch needs updating
It also currently throws exceptions on stop (if you have any other plugin installed) due to a bukkit service being registered with a null plugin, presumably within spark
I swear that this will lead to confusion as lucko/spark#424 didn't merge. If Spark keeps disabling after the update, it will get confused, This Pull was necessary to merge. As Spark will get disabled without warning. |
discussion: #10565
docs: PaperMC/docs#447
New configuration options (global configuration):