JSC tests takes a while to run (~45 mins). In case of failure, we retry the tests, and if failures are noticed in retry as well, then we un-apply the patch, build the ToT and run the tests again. All these steps might take a while to complete. Meanwhile the patch might get obsolete or r-. EWS should not continue to process such patches, as it would be wasteful. We should check the patch status before running any time-consuming step (like running tests, or building without patch). We did similar thing for layout and api tests in https://trac.webkit.org/changeset/252037/webkit
Created attachment 383612 [details] Patch
Comment on attachment 383612 [details] Patch Clearing flags on attachment: 383612 Committed r252485: <https://trac.webkit.org/changeset/252485>
All reviewed patches have been landed. Closing bug.
<rdar://problem/57228467>
Few examples where this change was useful in improving EWS efficiency (saving hours of processing times): https://ews-build.webkit.org/#/builders/1/builds/147 https://ews-build.webkit.org/#/builders/1/builds/246 https://ews-build.webkit.org/#/builders/1/builds/248 https://ews-build.webkit.org/#/builders/1/builds/249 https://ews-build.webkit.org/#/builders/1/builds/250