big +1! As Ahmet suggested, the IT-marked tests may need to have a different timeout. But other than that, I think this is great.
On Fri, Dec 20, 2019 at 9:39 AM Udi Meiri <[email protected]> wrote: > https://issues.apache.org/jira/browse/BEAM-9009 > > On Fri, Dec 20, 2019 at 6:18 AM Maximilian Michels <[email protected]> wrote: > >> +1 Good idea. We should also have this for Java if possible. >> >> On 20.12.19 02:59, Ahmet Altay wrote: >> > This sounds reasonable. Would this be configurable per-test if needed? >> > >> > On Thu, Dec 19, 2019 at 5:52 PM Udi Meiri <[email protected] >> > <mailto:[email protected]>> wrote: >> > >> > Looking at this console log >> > < >> https://builds.apache.org/job/beam_PreCommit_Python_Cron/lastCompletedBuild/timestamps/?time=HH:mm:ss&timeZone=GMT-8&appendLog&locale=en_US >> >, >> > it seems that some pytests got stuck (or slowed down considerably). >> > I'd like to put a 10 minute default timeout on all unit tests, using >> > the pytest-timeout <https://pypi.org/project/pytest-timeout/ >> > plugin. >> > >> >
