Hide Forgot
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): python-pthreading-1.3 This is the required interface: def notify(n=1): 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(). See https://docs.python.org/2/library/threading.html#threading.Condition.notify
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.