Discussion: sdk_max_attempts naming

[leggetter]

@alexluong I'm wondering if we should rename sdk_max_attempts to max_retry_attempts?

1. Why do we need to leak that this is a config option passed to the SDK?
2. We're missing that this is config about how many "retries" to attempt. So, how about adding retry to the name?

The release has gone out as a beta, so if we agree on this change, I'd suggest we rename it as a breaking change in the v0.5.0 full release.

[alexluong]

I think whichever name makes sense. I was trying to be consistent with the SDK. If you don't think it's helpful and rather treat TF as its own thing then we can certainly go with that.

The original idea was that if in the future there are other SDK options that we want to support, then we have the sdk_... prefix as a naming convention. But I also get where you're coming from as well.