Another stab at waitWithSemaphore -> Future<T>::wait()
Summary:
See
D1785572. Check out the new Future test and the commented portion of wait(Duration) for the fix
The test only fails a few times out of a hundred before the fix, but hasn't failed yet after
Test Plan: futures unit, wait for contbuild
Reviewed By: hans@fb.com
Subscribers: trunkagent, rushix, fbcode-common-diffs@, hero-diffs@, cold-storage-diffs@, adamsyta, zhuohuang, darshan, micha, folly-diffs@, lins, tingy, hannesr, jsedgwick
FB internal diff:
D1803526
Tasks:
5940008,
6059995
Signature: t1:
1803526:
1422309486:
3613c59a708ecac312d241723828763feb2a57aa