Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[improve][ci] Speed up execution of OTHER unit test group by removing -DtestReuseFork=false #17943

Merged
merged 1 commit into from
Nov 19, 2022

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Oct 5, 2022

Motivation

using -DtestReuseFork=false slows down test execution significantly.

Modifications

remove -DtestReuseFork=false for OTHER unit test group

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository: lhotari#100

@poorbarcode
Copy link
Contributor

/pulsarbot rerun-failure-checks

@Technoboy- Technoboy- closed this Nov 19, 2022
@Technoboy- Technoboy- reopened this Nov 19, 2022
@Technoboy- Technoboy- changed the title [improve][CI] Speed up execution of OTHER unit test group by removing -DtestReuseFork=false [improve][ci] Speed up execution of OTHER unit test group by removing -DtestReuseFork=false Nov 19, 2022
@codecov-commenter
Copy link

codecov-commenter commented Nov 19, 2022

Codecov Report

❗ No coverage uploaded for pull request base (master@4742b6f). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #17943   +/-   ##
=========================================
  Coverage          ?   47.36%           
  Complexity        ?    10468           
=========================================
  Files             ?      698           
  Lines             ?    67990           
  Branches          ?     7272           
=========================================
  Hits              ?    32205           
  Misses            ?    32196           
  Partials          ?     3589           
Flag Coverage Δ
unittests 47.36% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

@Technoboy- Technoboy- merged commit bc1e0cb into apache:master Nov 19, 2022
lifepuzzlefun pushed a commit to lifepuzzlefun/pulsar that referenced this pull request Dec 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-not-needed Your PR changes do not impact docs ready-to-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants