Thanks Łukasz.Should the solution be documented (in Beam testing guide ?) so that other performance tests can support manual triggering without affecting benchmark results in a similar manner ?- ChamOn Thu, Nov 22, 2018 at 4:03 AM Łukasz Gajowy <lukasz.gajowy@xxxxxxxxx> wrote:Hi all,
BEAM-6011 is now resolved. If any of you think your changes require such additional build/performance checks, now you can run them by posting a comment on Github (eg. "Run Direct Runner Nexmark Tests") to see if everything is fine before merging a PR.
Łukaszśr., 7 lis 2018 o 17:38 Andrew Pilloud <apilloud@xxxxxxxxxx> napisał(a):My concern is that PR triggered runs should not publish results to the same table as runs on master. Looks like you already covered that in the bug.There is also the issue that there is only one jenkins executor that can run the local runner jobs. This will only be a problem if the manual runs become frequent.AndrewOn Wed, Nov 7, 2018, 8:32 AM Łukasz Gajowy <lgajowy@xxxxxxxxxx wrote:Hi,
recent experience with Nexmark crashes made enabling Phrase Triggering in Nexmark suites even more urgent. If you have any opinions in this area feel free to share them.
Here's the link to a corresponding JIRA issue: https://issues.apache.org/jira/browse/BEAM-6011