Concerns about new get_uuid() implementation in aliyun-python-sdk-core 2.14.0 about aliyun-openapi-python-sdk HOT 3 CLOSED

1. This function is used to generate a unique nonce value for the openapi call, and we don't limit the format to satisfy the uuid. For the historical reasons, the function name may be confusing to readers.
2. This is a Python2 repository. Due to python GIL, only one thread can be run. If this GIL assumption does not hold, the randNum field can be used to reduce the collision rate.
3. We have tested the this design in tea-util in the real environment, we found that the collisions has been significantly reduced.

from aliyun-openapi-python-sdk.

Thank you for referring aliyun/tea-util#335 and the test results in production environments, though there's a minor issue with Python implementation (compared with Java/Golang implementation of same algorithm):

Due to python GIL, only one thread can be run.

A += 1 in Python would be compiled to multiple bytecodes (load, += and store):

>>> n = 0
>>> def f():
...  global n
...  n += 1
... 
>>> import dis
>>> dis.dis(f)
  1           0 RESUME                   0

  3           2 LOAD_GLOBAL              0 (n)
             14 LOAD_CONST               1 (1)
             16 BINARY_OP               13 (+=)
             20 STORE_GLOBAL             0 (n)
             22 LOAD_CONST               0 (None)
             24 RETURN_VALUE

And in Python 3, GIL would be taken by another thread if it has timed out (https://github.com/python/cpython/blob/65c285062ce2769249610348636d3d73153e0144/Python/ceval_gil.c#L20-L24C58. GIL is also preemptive in Python 2.x), thus n here is not an atomic variable. And unfortunately Python does not have built-in modules about atomics...

from aliyun-openapi-python-sdk.

感谢 @taoky 对这个改动的关注。作为生成 Nonce 的实现，目前看下来还是 OK 的。这个讨论就暂时先关掉了。

如果还有其余问题，请 re-open。

from aliyun-openapi-python-sdk.