These two lines create an async form of busy waiting, consuming all available CPU cycles to repeatedly check the condition. I would generally consider this not an acceptable Future
implementation outside of special circumstances (e.g. if the executor is known to only poll
after progress is made elsewhere such that it’s reasonable to check again).
2 Likes