AsyncSlimefun4 | 高性能异步化的Slimefun4分支

小张up · 发表于 2022-7-22 11:22:57

您尚未登录，立即登录享受更好的浏览体验！

您需要登录才可以下载或查看，没有帐号？注册(register)

x

本帖最后由小张up 于 2022-10-11 16:02 编辑

货运的异步在最新版被取消,其他功能无改变,原因请查阅置顶楼层

交流探讨? 加入我们的QQ群 423179929

感谢上游对SF提供的汉化,原帖地址:

https://www.mcbbs.net/thread-827594-1-1.html

此处下载:

https://github.com/HAPPYLAND-Dev/Slimefun4/actions

请勿使用

Thehrz · 发表于 2022-7-22 13:58:48

强烈不建议使用此Fork

Slimefun货运系统在达到一定规模时由于反复从容器遍历物品造成性能滞后

而之所以它需要反复从容器遍历物品是因为Bukkit 并没有 InventoryItemAddEvent 即物品被添加到一个Inventory的事件

这就导致Slimefun需要每隔一段时间从容器遍历物品判断被货运节点连接的容器内有没有符合要求的物品若有这个物品那就将这个物品放到另一个被货运节点连接的容器而一个原版大箱子为54格假设另一个箱子也是54格那它最多要遍历108次这只是二个输出输入货运节点+两个大箱子最大要遍历108次

而一个货运系统不可能只有一个节点它还要先遍历货运节点连接的箱子

基于此货运系统时间复杂度应为 O(n) 级但是由于容器最大格的存在其常数是很大的不可忽略

从容器遍历物品这个操作必须在主线程进行

我看到了你的代码

你将CargoNetworkTask整个变为异步执行

而CargoUtils那些的 insert withdraw 的方法

还是同步执行

但是 insert withdraw 就是整个货运系统最慢的地方啊

所以你整个异步执行又有什么用？虽然我暂时没有看出有线程不安全的地方但是存在这种风险是潜在的

而上面说到货运系统的性能滞后是因为规模大而规模大的情况下反复向Bukkit的任务调度器推任务这反而会造成性能损失

如果单靠你改个异步执行就能优化Slimefun货运系统那早就这么做了

依我看优化Slimefun货运系统就是BukkitAPI 添加 InventoryItemAddEvent 这样就免去每隔一段时间遍历但现阶段是没有的

综上 我认为你这个Fork 完全没有意义徒增风险 如果你将这个Fork PR 到Slimefun 主分支我相信会被那些老外直接Close

ZeroRev_ · 发表于 2022-7-22 11:39:08

不错的插件

全球4169 · 发表于 2022-7-22 11:51:58

真不错，有救了

超帅大帅 · 发表于 2022-7-22 12:28:42

MCBBS有你更精彩

pXiaoD · 发表于 2022-7-22 12:32:15

我觉得支持 SuperiorSkyblock 比支持 IridiumSkyblock 吸引力更大。

皓浩 · 发表于 2022-7-22 12:40:21

粘液科技必备插件，正在用很不错

皓浩 · 发表于 2022-7-22 12:40:43

希望作者大大持续更新

小张up · 发表于 2022-7-22 15:45:29

本帖最后由小张up 于 2022-7-22 15:58 编辑

Thehrz 发表于 2022-7-22 13:58
强烈不建议使用此Fork

Slimefun货运系统在达到一定规模时由于反复从容器遍历物品造成性能滞后

对于原版容器的异步确实不能异步，转到主线程执行是必须的，也必然会增加占用。
但是，您能否思考如下的一些问题：1.粘液科技的自动化，几乎完全依赖所有的机器，而不是原版的箱子
2.粘液科技的附属的那些储存桶，是可以异步访问的

我在我的服务器做了大量的测试和查看玩家的自动化设计，整个线只有1-2个原版容器，故因此我认为在访问原版储存时同步不会占用大量性能，于此同时异步所有机器的存取，绝对可以提高性能。

线程安全有问题，5分钟前我修复了
争议就在于货运访问原版库存，确实如果大量依赖原版的库存的话，占用不降反增，我稍后会在贴内着重提醒这一点帮助一些服主避雷

感谢您的提醒，能加我的QQ让我们来一同探讨吗？3296517911

小张up · 发表于 2022-7-22 15:50:44

pXiaoD 发表于 2022-7-22 12:32
我觉得支持 SuperiorSkyblock 比支持 IridiumSkyblock 吸引力更大。

好提议，有空就去做
主要是没有现成的测试服务器能用emm

Thehrz · 发表于 2022-7-22 16:38:32

小张up 发表于 2022-7-22 15:45
对于原版容器的异步确实不能异步，转到主线程执行是必须的，也必然会增加占用。
但是，您能否思考如下的一 ...

我不了解Slimefun附属

你重置的是Slimefun 你不能控制用户用不用附属插件

因此这个问题不应该交流关于附属插件

你可以先看一下ChestMenu类 (Slimefun 机器Menu实现)

这个类本质是Bukkit API Inventory 封装

Inventory 类不能用异步访问而且他维护的items List 的实现也仅仅是ArrayList 是线程不安全List实现

请问你是怎么得出机器容器是可以异步访问的

小张up · 发表于 2022-7-22 17:02:20

本帖最后由小张up 于 2022-7-22 17:08 编辑

Thehrz 发表于 2022-7-22 16:38
我不了解Slimefun附属

你重置的是Slimefun 你不能控制用户用不用附属插件

按照你的逻辑，货运是打开机器页面模拟玩家操作不成？

仅仅为原版容器，才会切到主线程执行

Thehrz · 发表于 2022-7-22 17:11:35

小张up 发表于 2022-7-22 17:02
按照你的逻辑，货运是打开机器页面模拟玩家操作不成？

我并没有告诉你Slimefun进行过这种操作

Inventory 是线程不安全类或者说carftbukkit包下的类都是线程不安全类

这些类没有任何锁、同步机制因此造成了线程不安全

为什么你理解成要"模拟玩家"操作 Inventory ？

小张up · 发表于 2022-7-22 17:13:46

Thehrz 发表于 2022-7-22 17:11
我并没有告诉你Slimefun进行过这种操作

Inventory 是线程不安全类或者说carftbukkit包下的类都是线程不 ...

我当然知道，但你说的

反复向Bukkit的任务调度器推任务

是什么意识

蟹蒙老鸽 · 发表于 2022-7-22 17:17:35

本帖最后由蟹蒙老鸽于 2022-7-22 17:31 编辑

厉害，能有想法去实践已经是难得可贵的了。其实推荐既然你要做独立出去的一个粘液分支：
1. 可以把粘液机器变原版化问题也一并解决，对于粘液原版化只需要把机器底子改成容器方块可以大大降低变原版可能性（我服务器测试这样做了后玩家几乎没有出现机器变原版的问题了

2. 可以尝试把Mysql融合到粘液里面去，我记得有一个开源附属做到了

帐号		自动登录	找回密码
密码			注册(register)

拯救MCBBS：论坛页面归档列表

AsyncSlimefun4 _ 高性能异步化的Slimefun4分支 - 服务端插件 - Minecraft(我的世界)中文论坛 -【】mcbbs5.3】.html

[重制插件] AsyncSlimefun4 | 高性能异步化的Slimefun4分支

您尚未登录，立即登录享受更好的浏览体验！

评分

评分

评分

[重制插件] AsyncSlimefun4 | 高性能 异步化的Slimefun4分支

您尚未登录，立即登录享受更好的浏览体验！

评分

评分

评分

[重制插件] AsyncSlimefun4 | 高性能异步化的Slimefun4分支