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.

Sign up for GitHub

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

Jump to bottom

Release lock client in KTI #6075

Merged
merged 2 commits into from Dec 14, 2015
Merged

Release lock client in KTI #6075

merged 2 commits into from Dec 14, 2015

Commits on Dec 10, 2015

  1. KernelTransactionImplementation doesn't reference Locks.Client 

    since those instances are hard wired to a specific Locks instance. Other
similar references are always kept on the service level, like
CommitProcess, where there's a proxy in between which switches when role
switches.

This commit undos parts of #5817 because of the issue described in #5996
and because this solution where Locks is referenced conforms with
other types of references and is a simpler fix to the original problem.
    @tinwelint @lutovich
    tinwelint authored and lutovich committed Dec 10, 2015
    Copy the full SHA
    1eb32a0 View commit details
    Browse the repository at this point in the history

  2. Added test for GitHub issue #5996

    @lutovich
    lutovich committed Dec 10, 2015
    Copy the full SHA
    99abe5c View commit details
    Browse the repository at this point in the history