Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@cunheise
Copy link

add igbinary serializer handler support, currently the project session serializer handler only support php serializer, add igbinary serializer support

@fuzqing
Copy link
Contributor

fuzqing commented Aug 31, 2025

这是一个很好的 PR,支持 igbinary 确实可以带来性能和存储上的优势。不过既然引入了这个功能,建议将代码库中所有 serialize 和 unserialize 的地方都统一替换掉,这样可以保持代码的一致性,也能让 igbinary 的优势在更多地方体现出来。

@walkor
Copy link
Owner

walkor commented Sep 2, 2025

这个是一个破坏性更新,无法合并。
另外需要测试 igbinary 到底快多少。如果只是快一些,其实相比磁盘等io等待优化效果微乎其微,但是带来了其它例如复杂性、稳定性、破坏性等副作用,不如保持现状 。

@cunheise
Copy link
Author

cunheise commented Sep 2, 2025

破坏性更新?代码里面有判断如果igbinary lib有安装,并且php.ini session.handler 设置为了igbinary 才会使用 igbinary做为session handler,默认还是使用php serializer

@walkor
Copy link
Owner

walkor commented Sep 2, 2025

抱歉,没注意到那个判断

@walkor walkor merged commit bbb01ba into walkor:master Sep 2, 2025
24 checks passed
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.

3 participants