You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
on embedded derby storage on standalone model ,config cas update not work.
Describe the bug
com.alibaba.nacos.config.server.service.repository.PersistService#updateConfigInfoCas
where current md5 is not match ,config cas update method should return update fail.
ExternalConfigInfoPersistServiceImpl works ,but EmbeddedConfigInfoPersistServiceImpl works.
Expected behavior
where BaseDatabaseOperate update method executing modified requests, it should check the number of rows affected
Actually behavior
it seems that BaseDatabaseOperate update do not recognized the update result value ,so when cas check fail ,updateConfigInfoCas method return true still.
If someone interested in this issue ,create pull request to solve it.
The text was updated successfully, but these errors were encountered:
* fix issue 9943
* add some description
* add some log for roll back transaction
* add some log for roll back transaction
* set default rollBackOnUpdateFail false.
* format style
* fix some bug
on embedded derby storage on standalone model ,config cas update not work.
Describe the bug
com.alibaba.nacos.config.server.service.repository.PersistService#updateConfigInfoCas
where current md5 is not match ,config cas update method should return update fail.
ExternalConfigInfoPersistServiceImpl works ,but EmbeddedConfigInfoPersistServiceImpl works.
Expected behavior
where BaseDatabaseOperate update method executing modified requests, it should check the number of rows affected
Actually behavior
it seems that BaseDatabaseOperate update do not recognized the update result value ,so when cas check fail ,updateConfigInfoCas method return true still.
If someone interested in this issue ,create pull request to solve it.
The text was updated successfully, but these errors were encountered: