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

force cache the docker layer #4331

Merged
merged 1 commit into from
Jun 17, 2022
Merged

Conversation

HarrisChu
Copy link
Contributor

What type of PR is this?

  • bug
  • feature
  • enhancement

What problem(s) does this PR solve?

Issue(s) number:

Description:

force cache the docker layer.
the layer is about 20GB, and need another 20min to export the layer.
after then each job would need 5min with the cache layer.

How do you solve it?

Special notes for your reviewer, ex. impact of this fix, design document, etc:

Checklist:

Tests:

  • Unit test(positive and negative cases)
  • Function test
  • Performance test
  • N/A

Affects:

  • Documentation affected (Please add the label if documentation needs to be modified.)
  • Incompatibility (If it breaks the compatibility, please describe it and add the label.)
  • If it's needed to cherry-pick (If cherry-pick to some branches is required, please label the destination version(s).)
  • Performance impacted: Consumes more CPU/Memory

Release notes:

Please confirm whether to be reflected in release notes and how to describe:

ex. Fixed the bug .....

@HarrisChu HarrisChu added the ready-for-testing PR: ready for the CI test label Jun 17, 2022
@HarrisChu
Copy link
Contributor Author

@Sophie-Xie Sophie-Xie merged commit 2c33d7a into vesoft-inc:master Jun 17, 2022
@Sophie-Xie Sophie-Xie added the cherry-pick-v3.2 PR: need cherry-pick to this version label Jun 17, 2022
nebula-bots pushed a commit that referenced this pull request Jun 23, 2022
Sophie-Xie pushed a commit that referenced this pull request Jun 27, 2022
Sophie-Xie added a commit that referenced this pull request Jun 27, 2022
* force cache the docker layer (#4331)

* check god role exist when meta init (#4330)

* check god role exist when meta init

* return error if kv fail

Co-authored-by: Doodle <13706157+critical27@users.noreply.github.com>

* Fix object pool mtsafe. (#4332)

* Fix object pool mtsafe.

* Fix lock.

* Fixed web service crash (#4334)

Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>

* Fix get edges transform rule. (#4328)

1. Input/Ouput variables.
2. Keep column names of Limit same with input plan node.

Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>

* fix rc docker (#4336)

* add lock (#4352)

* fix map concurrency issue (#4344)

* fix mutex in map

* add test

* move the order

Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>

* add stats under index conditions (#4353)

Co-authored-by: Harris.Chu <1726587+HarrisChu@users.noreply.github.com>
Co-authored-by: jimingquan <mingquan.ji@vesoft.com>
Co-authored-by: Doodle <13706157+critical27@users.noreply.github.com>
Co-authored-by: shylock <33566796+Shylock-Hg@users.noreply.github.com>
Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
Co-authored-by: panda-sheep <59197347+panda-sheep@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick-v3.2 PR: need cherry-pick to this version ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants