mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Mahler <>
Subject Re: Review Request 72405: Added support for read readiness polling on Windows.
Date Fri, 24 Apr 2020 18:04:37 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated April 24, 2020, 6:04 p.m.)

Review request for mesos, Andrei Sekretenko and Greg Mann.


* Fixed an ifdef to use `ENABLED_LIBWINIO` instead of `__WINDOWS__`.

Repository: mesos


This uses the zero byte read trick on Windows in order to
implement read readiness polling on Windows.

The documentation for io::read has also been updated to explain
how zero byte reads are treated consistently across POSIX and

Write readiness polling is left unimplemented since it's less
straightforward. A good explanation of the Windows landscape
for polling is here:

A library providing an epoll like interface on Windows can
be found here:

Diffs (updated)

  3rdparty/libprocess/include/process/io.hpp 468b3626e0121bc380ea592f485f3cfe9796d0fe 
  3rdparty/libprocess/src/io.cpp 7288d5fa1c79f5d19a9369e6431dc98d92307868 
  3rdparty/libprocess/src/io_internal.hpp 09bfa9a386a3eb89a987eea18e08144a50aaf5c7 
  3rdparty/libprocess/src/windows/io.cpp 90e780d4d3eae7af71ee39ff99105147992e968d 




Added a test in subsequent patch.


Benjamin Mahler

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message