TapTap/FastSync
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Static compression level wastes CPU on incompressible files #110

New Issue
Closed
opened 2026-07-21 16:20:27 +02:00 by TapTap · 1 comment
TapTap commented 2026-07-21 16:20:27 +02:00
Owner
Copy Link
Copy Source

Description

Compression level is set once (default 5). Already-compressed files (jpg, mp4, zstd) waste CPU cycles being recompressed pointlessly.

Location

src/shared/compression.c

Suggested Fix

Skip compression for files with known incompressible extensions (e.g., .jpg, .mp4, .zip, .gz, .zst). Optionally add heuristics: compress a small sample and check ratio before committing.

Severity

medium

Category

performance

Sub-Agent

perf-analyst (PA-5)

This issue was automatically generated by the issue-creator agent.

## Description Compression level is set once (default 5). Already-compressed files (jpg, mp4, zstd) waste CPU cycles being recompressed pointlessly. ## Location src/shared/compression.c ## Suggested Fix Skip compression for files with known incompressible extensions (e.g., `.jpg`, `.mp4`, `.zip`, `.gz`, `.zst`). Optionally add heuristics: compress a small sample and check ratio before committing. ## Severity medium ## Category performance ## Sub-Agent perf-analyst (PA-5) --- _This issue was automatically generated by the issue-creator agent._
TapTap commented 2026-07-29 18:27:33 +02:00
Author
Owner
Copy Link
Copy Source

Fixed in PR #148 — merged into main on 2026-07-29.

Fixed in PR #148 — merged into main on 2026-07-29.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Defect or bug
 enhancement
Feature or improvement
 memory
Memory safety issue
 needs-triage
Needs human review
 performance
Performance issue
 quality
Code quality improvement
 security
Security vulnerability
 threading
Threading or concurrency
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
TapTap
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: TapTap/FastSync#110
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?