Skip to content

optimize: improve pull process and switch the sha256 library to simd#103

Merged
gaius-qi merged 1 commit intomainfrom
optimize/sha256
Mar 4, 2025
Merged

optimize: improve pull process and switch the sha256 library to simd#103
gaius-qi merged 1 commit intomainfrom
optimize/sha256

Conversation

@chlins
Copy link
Member

@chlins chlins commented Mar 4, 2025

This pull request includes several changes to improve the handling of blob storage and signal handling in the application. The most important changes include modifications to the PushBlob method to accept an additional ocispec.Descriptor parameter, updates to the go.mod dependencies, and the addition of signal handling for graceful shutdown.

Improvements to blob storage handling:

Dependency updates:

  • go.mod: Added github.com/minio/sha256-simd and github.com/klauspost/cpuid/v2 as dependencies, and moved github.com/sirupsen/logrus to indirect dependencies. [1] [2] [3]

Signal handling for graceful shutdown:

  • cmd/root.go: Added signal handling to gracefully shut down the application on receiving an interrupt signal (SIGTERM). [1] [2]

These changes collectively enhance the robustness and maintainability of the application by improving blob storage operations and ensuring a graceful shutdown process.

…6-simd

Signed-off-by: chlins <chlins.zhang@gmail.com>
@chlins chlins force-pushed the optimize/sha256 branch from 24e6eb8 to 7f0cbe3 Compare March 4, 2025 06:44
@chlins chlins added the enhancement New feature or request label Mar 4, 2025
Copy link
Member

@gaius-qi gaius-qi left a comment

Choose a reason for hiding this comment

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

LGTM

@gaius-qi gaius-qi merged commit bf0c4a4 into main Mar 4, 2025
5 checks passed
@gaius-qi gaius-qi deleted the optimize/sha256 branch March 4, 2025 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants