BaseAPIClient: add docstring mentioning class is not thread-safe

[risicle]

FWIW this isn't necessarily a bad thing, it's just something users should be aware of if running in a multi-threaded context.

I guess we should also add this information to the user-facing documentation with a hint about what it means and what users can do about it.