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

refactor: make LocalCache not use synchronized to detect recursive loads (#6845) #6851

Closed
wants to merge 3 commits into from
Closed

refactor: make LocalCache not use synchronized to detect recursive loads (#6845) #6851

wants to merge 3 commits into from

Commits on Nov 29, 2023

  1. test: recursive load fail-fast (#6845)

    Christian Ortlepp authored and Christian Ortlepp committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    cb045df View commit details
    Browse the repository at this point in the history

  2. test: recursive load with proxy (#6845)

    Christian Ortlepp authored and Christian Ortlepp committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    e80103f View commit details
    Browse the repository at this point in the history

  3. refactor: thread reference in LoadingValueReference to detect recursi… 

    …on (#6845)

- replaces the previous synchronized implementation to make it work gracefully with VirtualThreads
- remember the thread that created a LoadingValueReference to later determine whether the "loader" is the same thread as the waiting thread, and we are therefore inside a deadlock
    Christian Ortlepp authored and Christian Ortlepp committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    498baf1 View commit details
    Browse the repository at this point in the history