Conversation
Lumine1909
left a comment
There was a problem hiding this comment.
感谢你的pr!
鉴于Linear格式是一个极度危险的实验性功能 我不建议你继续使用它(
并且 这个文件的代码质量在我看来是leaves项目中最差的 应该整体重构(
|
改好了,麻烦您在看一下 |
|
改好了,麻烦您再看一下。实在是抱歉打扰了。喵(qwq) |
|
改完了,麻烦您再看一眼QAQ |
|
还有感觉那个public boolean regionFileOpen = false;是不是要改成public volatile boolean regionFileOpen = false;来解决一下多线程的可见性来着(刚想起来) |
|
喵,其实还有想要优化的地方,不知道是否是应该打开新的pr还是在这里修改 |
|
想做可以一起做了然后把pr改名城linear重构 大概是这样的 顺便可以转为 draft pr |
|
记得看一下 lhx28#1 qwq |
|
麻烦再看一下,打扰了qwq |
|
已按要求全部修改完啦,麻烦再看下qwq |
|
qwq,刚才发现按照建议给 close 加 synchronized 去掉读写锁的话可能会出现队列中最后修改丢失的问题 |
|
已经把 close 的锁逻辑修复好啦,换回安全的版本了qwq,麻烦再看下~ |
|
先等会 |
|
实在是抱歉,那个其实应该还有些遗留问题(linear的(因为之前重构只有那个线程上的,其实在观察后发现还有内存溢出,硬盘写入过量的数据导致的问题))(很复杂的)需要修改,我建议是在我的那个测试服务器上跑上一天再提交 |
|
实在是麻烦您了 |
|
很抱歉在这个时候打断您 |
|
那个,测试的结果大概明天中午15:00出,到时候再说吧(如果您方便的话)实在是抱歉了 |
|
没事,您先测,我暂时没有看出问题。 |
|
好哒qwq!那我统一改 ms 单位,谢谢您的建议~ |
s-yh-china
left a comment
There was a problem hiding this comment.
很抱歉隔着一周才重新进行审查,我认为您的修改应该已经没有问题,这项工作是值得肯定的,剩下的只剩一些细微的格式化工作,您可以在 idea 中直接对 LinearRegionFile 进行一次格式化操作,其应该有少量更改内容。
真正值得注意问题是,您的部分提交似乎缺少了签名,这可能是github网页版的漏洞,但这仍然会使得我们无法对您的pr进行合并。如果可以,您可以尝试在本地对此pr的相关提交记录压缩后一并重新签名,而后只需要进行一次强制提交就可以了。
4f38723 to
8fe385d
Compare
|
抱歉Lumine1909,您的合并记录好像被压到一起了QAQ,我真的不怎么会用github |
还是没有签名( |
8fe385d to
3f8ce07
Compare
|
非常抱歉,麻烦您看下~QAQ |







服务器使用的linear v2作为储存,没有打开虚拟线程(打开了以后会导致内存爆得更快)(JVM 虚拟线程调度器会强持有所有虚拟线程实例,内存永远回收不了)。由于每打开一个区块文件(.linear)都会拉起一个新的线程,导致服务器产生大量的linear io线程


所以把每个文件加载都启动一个线程改为了全局线程池
周末打扰您真的麻烦了