Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: initialize transactionMu in NewAdapterByDBUseTableName #237

Merged
merged 1 commit into from
May 31, 2024

Commits on May 30, 2024

  1. fix: initialize all transactionMu in Adapter constructors

    constructor NewAdapterByDBUseTableName should initialize transactionMu,
    otherwise it will ~~panic~~ blocked due to that cas when calling Transaction
    
    Besides, a few other constructors also depend on NewAdapterByDBUseTableName,
    if transactionMu is not initialized in NewAdapterByDBUseTableName,
    it will blocked as well.
    
    Besides, why not consider use sync.Once to initialize transactionMu?
    yuikns committed May 30, 2024
    Configuration menu
    Copy the full SHA
    5c84803 View commit details
    Browse the repository at this point in the history