Before going stable, there's a chance to review and align the timeout values:
- In traces, export and flush timeouts default to 30 seconds, whereas in metrics they default to 10 seconds. In logging most places use 10, but one-two use 30 seconds instead. They should all get the same value.
- In some methods,
timeout can receive a None value where in other places it can only get an int. Whatever decision is taken, all methods should align to that:
def force_flush(self, timeout_millis: int | None = None):
# VS
def force_flush(self, timeout_millis:int = None):
Before going stable, there's a chance to review and align the timeout values:
timeoutcan receive aNonevalue where in other places it can only get an int. Whatever decision is taken, all methods should align to that: