Skip to content

Conversation

@yanglimingcn
Copy link
Contributor

添加mysql的事务功能。
issue #209

@yanglimingcn
Copy link
Contributor Author

yanglimingcn commented Apr 11, 2019

yanglimingcn@56e76f4
@jamesge 麻烦帮我评审一下,这次提交,把pipeline的方式换成correlation_id,是不是更合理些?
这里是通过判断_auth_context是否为NULL,来判断是否需要做权限校验的,做完权限校验,再把_auth_context置为NULL,这时候就会报错:
Impossible! This function is supposed to be called only once when verification succeeds in server side only once when verification succeeds in server side
这里需要把void Controller::set_auth_context(const AuthContext* ctx) 这个函数改一下,把里面的_auth_context != NULL注释掉。

@yanglimingcn yanglimingcn force-pushed the mysql_transaction branch 2 times, most recently from 2fac1a9 to 260ef8a Compare April 20, 2019 21:38
@yanglimingcn
Copy link
Contributor Author

支持single模式,使用事务功能将不支持single模式。添加了压测脚本,更新了文档。

@yanglimingcn yanglimingcn force-pushed the mysql_transaction branch 3 times, most recently from 2ca8885 to 043346f Compare April 24, 2019 09:42
@yanglimingcn
Copy link
Contributor Author

#760

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.

1 participant