Skip to content

cherry pick #2017 #2020 #2051 to release-4.0#2073

Merged
ti-srebot merged 4 commits intopingcap:release-4.0from
solotzg:opt-ccache-4.0
Jun 3, 2021
Merged

cherry pick #2017 #2020 #2051 to release-4.0#2073
ti-srebot merged 4 commits intopingcap:release-4.0from
solotzg:opt-ccache-4.0

Conversation

@solotzg
Copy link
Contributor

@solotzg solotzg commented Jun 3, 2021

What is changed and how it works?

What's Changed:

  • accelerate compiling speed by ccache
  • compile without debug info during ci build
  • use ccache to accelerate speed
  • remove docker build in ci

Related changes

  • PR to update pingcap/docs/pingcap/docs-cn:
  • Need to cherry-pick to the release branch:

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Release note

  • No release note

solotzg added 4 commits June 2, 2021 23:26
* update default config & show message about ccache
* disable ccache in ci-build or release-build

Signed-off-by: Zhigao Tong <tongzhigao@pingcap.com>
* compile without debug info during ci build
* use `ccache` to accelerate speed
* remove docker build in ci

Signed-off-by: Zhigao Tong <tongzhigao@pingcap.com>
# Conflicts:
#	tests/docker/run.sh
* accelerate compiling speed by ccache

Signed-off-by: Zhigao Tong <tongzhigao@pingcap.com>
Copy link
Contributor

@zanmato1984 zanmato1984 left a comment

Choose a reason for hiding this comment

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

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Jun 3, 2021
@solotzg
Copy link
Contributor Author

solotzg commented Jun 3, 2021

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Jun 3, 2021
@ti-srebot
Copy link
Collaborator

/run-all-tests

@ti-srebot ti-srebot merged commit ad87508 into pingcap:release-4.0 Jun 3, 2021
@solotzg solotzg deleted the opt-ccache-4.0 branch June 4, 2021 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants