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

Run tests in parallel in travis-ci #1569

Merged
merged 1 commit into from Nov 29, 2018
Merged

Run tests in parallel in travis-ci #1569

merged 1 commit into from Nov 29, 2018

Conversation

xnull
Copy link
Collaborator

@xnull xnull commented Nov 28, 2018

Overview

Description:
Run unit and integration tests on travis-ci in parallel which reduces total time to build and test a PR.
Setup Maven wrapper to provide better automation and make builds more stable.

Why should this be merged:
Improve tests performance in travis

Related issue(s) (if applicable): #

Checklist (Definition of Done):

  • There are no TODOs left in the code
  • Coding conventions (e.g. for logging, unit tests) have been followed
  • Change is covered by automated tests
  • Public API has Javadoc

@codecov
Copy link

codecov bot commented Nov 28, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff            @@
##             master   #1569   +/-   ##
========================================
  Coverage          ?      0%           
========================================
  Files             ?     282           
  Lines             ?   12206           
  Branches          ?    1201           
========================================
  Hits              ?       0           
  Misses            ?   12206           
  Partials          ?       0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ea6d065...fd4d60a. Read the comment docs.

@@ -0,0 +1,110 @@
/*
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A review is not required, it's a middleware, it was generated by maven wrapper

@@ -0,0 +1 @@
distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.6.0/apache-maven-3.6.0-bin.zip
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A review is not required, it's a middleware, it was generated by maven wrapper

@@ -0,0 +1,286 @@
#!/bin/sh
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A review is not required, it's a middleware, it is a maven wrapper

@@ -0,0 +1,161 @@
@REM ----------------------------------------------------------------------------
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A review is not required, it's a middleware, it is a maven wrapper for windows

@xnull xnull force-pushed the testperformanceparallel branch 4 times, most recently from df5cf88 to a4c0703 Compare November 28, 2018 08:14
@xnull
Copy link
Collaborator Author

xnull commented Nov 28, 2018

Total time 16 min vs 50 earlier

image

@no2chem
Copy link
Member

no2chem commented Nov 28, 2018

Very nice, @xnull . I'll leave it to someone who has more recently touched the code to review it.

@Maithem
Copy link
Contributor

Maithem commented Nov 29, 2018

Amazing. Thanks.

@Maithem Maithem merged commit c2326c2 into master Nov 29, 2018
@Maithem Maithem deleted the testperformanceparallel branch November 29, 2018 00:14
lixinchengdu pushed a commit to lixinchengdu/CorfuDB that referenced this pull request Jan 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants