Skip to content

Revert "Revert "Compress logs and profile events in the native protocol""#83586

Merged
alexey-milovidov merged 7 commits intomasterfrom
revert-83490-revert-82535-compression-logs-profile-events
Oct 15, 2025
Merged

Revert "Revert "Compress logs and profile events in the native protocol""#83586
alexey-milovidov merged 7 commits intomasterfrom
revert-83490-revert-82535-compression-logs-profile-events

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

@alexey-milovidov alexey-milovidov commented Jul 10, 2025

Reverts #83490

Changelog category (leave one):

  • Performance Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Compress logs and profile events in the native protocol. On clusters with 100+ replicas, uncompressed profile events take 1..10 MB/sec, and the progress bar is sluggish on slow Internet connections. This closes #82533.

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Jul 10, 2025

Workflow [PR], commit [b75dc90]

Summary:

job_name test_name status info comment
Stateless tests (arm_binary, sequential) failure
03174_exact_rows_before_aggregation FAIL
Integration tests (amd_tsan, 1/6) error
Performance Comparison (amd_release, master_head, 3/3) failure
Start failure

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jul 10, 2025
@alexey-milovidov alexey-milovidov self-assigned this Oct 15, 2025
@alexey-milovidov alexey-milovidov merged commit f2cd763 into master Oct 15, 2025
118 of 123 checks passed
@alexey-milovidov alexey-milovidov deleted the revert-83490-revert-82535-compression-logs-profile-events branch October 15, 2025 14:01
@robot-clickhouse robot-clickhouse added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Compress logs and profile events in the native protocol

2 participants