Description of problem:
pthreading.Condition.notify does not provide the n argument that was added in Python 2.7. This make prevent using this class as a drop in replacement fro the builtin threading.Condition class.
Version-Release number of selected component (if applicable):
This is the required interface:
When calling with n>1, this should wake up n waiters, but the Python
documentation warn that " it’s not safe to rely on this behavior. A future, optimized implementation may occasionally wake up more than n threads."
So we may choose to implement it ny invoking notify_all().
We could just do a for loop calling notify() though I don't understand why make this part of the interface. Could easily be done by the caller. Classic python stdlib over-design
Bug tickets must have version flags set prior to targeting them to a release. Please ask maintainer to set the correct version flags and only then set the target milestone.
Yaniv, is anyone working on it - or are we going to?
with vdsm current use cases in pthreading I don't see any need to align the interface and publish new version of pthreading for ovirt
Nir, please tell your opinion and change status if needed.
I agree with Yaniv, we just need to document this difference, in case someone try
to use pthreading in another context.