youngoli commented on a change in pull request #15094:
URL: https://github.com/apache/beam/pull/15094#discussion_r711314088
##########
File path: sdks/go/test/integration/io/xlang/kafka/jar.go
##########
@@ -13,13 +13,18 @@
// See the License for the specific language governing permissions and
// limitations under the License.
+// Exclude OSes which do not support the timeout command or syscall.SIGTERM.
Review comment:
AFAIK there's no way to implement timeout programmatically. Its purpose
is to shut down the container if the test binary doesn't get a chance to do so.
Checking for the timeout command sounds good though, I'll see if there's an
OS-independent way to do that.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
