Improve thread safety docs for fields in ServerImpl
Open
Type: Documentation
- Dominant language
- Java
- Stars
- 12.1k
- Forks
- 4k
- Avg merge
- 2d 17h
- Merged PRs (30d)
- 37
Description
Split out of https://github.com/grpc/grpc-java/pull/2032#discussion_r70340687. `usingSharedExecutor` and `executor` aren't using `GuardedBy` semantics. It isn't clear to @carl-mastrangelo that the current code is valid. Improve the docs/code to correct/make the correctness more clear.
Contributor guide
Assessment
This issue has not been assessed yet.