/backport to release/6.0

Started backporting to release/6.0: https://github.com/dotnet/aspnetcore/actions/runs/1218001947

These APIs suck

@davidfowl I kind of agree. They don't follow SRP.

Would it maybe be worth a new implementation where:

• Load could be public and throw on failure like LoadAsync does (maybe could extend ISession with a DIM?)
• Methods that currently call Load could throw InvalidOperationException if !_loaded to enforce the usage pattern of calling Load or LoadAsync first.
• IsAvailable might not be needed, since load and commit failures (Exceptions) would be the indicators of availability?

Those are just some thoughts I had. There might be other concerns or limitations I haven't thought of. Just throwing it out there 🙂.

Hi @csturm83. It looks like you just commented on a closed PR. The team will most probably miss it. If you'd like to bring something important up to their attention, consider filing a new issue and add enough details to build context.