♻️ Resolve busy-loop on worker

[rinx]

Signed-off-by: Rintaro Okamura rintaro.okamura@gmail.com

Description:

Resolve worker busy loop.
And fixed metrics definitions.

Related Issue:

compressor uses CPU too much, i've profiled using pprof.

it shows worker.queue.pop occupies the large amount of CPU.

How Has This Been Tested?:

On my k8s cluster.

Environment:

• Golang Version: 1.14
• Docker Version: 19.03.5
• Kubernetes Version: 1.17.3
• NGT Version: 1.9.1

Types of changes:

• Bug fix [type/bug]
• New feature [type/feature]
• Add tests [type/test]
• Security related changes [type/security]
• Add documents [type/documentation]
• Refactoring [type/refactoring]
• Update dependencies [type/dependency]
• Update benchmarks and performances [type/bench]
• Update CI [type/ci]

Changes to Core Features:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your core changes, as applicable?
• Have you successfully ran tests with your changes locally?

Checklist:

• I have read the CONTRIBUTING document.
• I have checked open Pull Requests for the similar feature or fixes?
• I have added tests and benchmarks to cover my changes.
• I have ensured all new and existing tests passed.
• I have commented my code, particularly in hard-to-understand areas
• I have updated the documentation accordingly.

[pull-assistant]

Best reviewed: commit by commit

---

Optimal code review plan

     ♻️ Resolve busy-loop on worker

     📈 Fix metrics definitions

     🤖 Update license headers and formatting go codes

Powered by Pull Assistant. Last update 58d9ed3 ... bba700c. Read the comment docs.

[rinx]

It's ready for review. @kpango
The compressor metrics now returns actual values.

[kpango]

LGTM
/format
/approve

[vdaas-ci]

[FORMAT] Updating license headers and formatting go codes triggered by kpango.

[vdaas-ci]

[APPROVED] This PR is approved by kpango.