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.

Sign up for GitHub

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

Jump to bottom

grpc-compression-type should affect TiKV's response to TiDB #17176

Closed
ekexium opened this issue Jun 21, 2024 · 1 comment · Fixed by #17180
Closed

grpc-compression-type should affect TiKV's response to TiDB #17176

ekexium opened this issue Jun 21, 2024 · 1 comment · Fixed by #17180
Labels
affects-5.4 affects-6.1 affects-6.5 affects-7.1 affects-7.5 affects-8.1 report/customer Customers have encountered this bug. type/enhancement Type: Issue - Enhancement

Comments

@ekexium
Copy link
Contributor

ekexium commented Jun 21, 2024 •
edited
Loading

Bug Report

grpc-compression-type doesn't affect TiKV's response to TiDB.
Corresponding args may be missing here

tikv/src/server/server.rs

Line 96 in 8e1f26b

let channel_args = ChannelBuilder::new(Arc::clone(&env))

What version of TiKV are you using?

v6.5.2

What operating system and CPU are you using?

Doesn't matter

Steps to reproduce

Run one large SELECT statement. Monitor TiKV's network traffic.

What did you expect?

Compression setting works.

What did happened?

Compression doesn't work.
@ekexium ekexium added the type/bug Type: Issue - Confirmed a bug label Jun 21, 2024
@ekexium ekexium mentioned this issue Jun 24, 2024
Merged
9 tasks
@ekexium ekexium added severity/moderate type/enhancement Type: Issue - Enhancement and removed type/bug Type: Issue - Confirmed a bug severity/moderate labels Jun 24, 2024
ti-chi-bot bot added a commit that referenced this issue Jun 24, 2024
@ekexium @ti-chi-bot
grpc: set compression args for TiKV service (#17180)
a1f05d6 
close #17176

Set compression arguments for TiKV service. The compression arguments are loaded from TiKV config.
It will affect TiKV's response to TiDB.

Signed-off-by: ekexium <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@zhangjinpeng87 zhangjinpeng87 mentioned this issue Jun 26, 2024
Merged
18 tasks
@ekexium ekexium changed the title grpc-compression-type doesn't affect TiKV's response to TiDB grpc-compression-type should affect TiKV's response to TiDB Jun 27, 2024
3AceShowHand pushed a commit to 3AceShowHand/tikv that referenced this issue Jun 27, 2024
@ekexium @ti-chi-bot @3AceShowHand
grpc: set compression args for TiKV service (tikv#17180)
6cf8ba4 
close tikv#17176

Set compression arguments for TiKV service. The compression arguments are loaded from TiKV config.
It will affect TiKV's response to TiDB.

Signed-off-by: ekexium <[email protected]>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Signed-off-by: 3AceShowHand <[email protected]>
@seiya-annie
Copy link

/found customer

@ti-chi-bot ti-chi-bot bot added the report/customer Customers have encountered this bug. label Jun 27, 2024
@ti-chi-bot ti-chi-bot mentioned this issue Jul 11, 2024
Merged
9 tasks
ti-chi-bot bot pushed a commit that referenced this issue Jul 11, 2024
@ti-chi-bot @ekexium
grpc: set compression args for TiKV service (#17180) (#17263)
bd8245e 
close #17176

Set compression arguments for TiKV service. The compression arguments are loaded from TiKV config.
It will affect TiKV's response to TiDB.

Signed-off-by: ekexium <[email protected]>

Co-authored-by: ekexium <[email protected]>
This was referenced Jul 26, 2024
Merged
Merged
Merged
ti-chi-bot bot added a commit that referenced this issue Jul 26, 2024
@ti-chi-bot @ekexium @ti-chi-bot
grpc: set compression args for TiKV service (#17180) (#17313)
3ffa98d 
close #17176

Set compression arguments for TiKV service. The compression arguments are loaded from TiKV config.
It will affect TiKV's response to TiDB.

Signed-off-by: ekexium <[email protected]>

Co-authored-by: ekexium <[email protected]>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ti-chi-bot bot pushed a commit that referenced this issue Jul 26, 2024
@ti-chi-bot @ekexium
grpc: set compression args for TiKV service (#17180) (#17314)
46e7b78 
close #17176

Set compression arguments for TiKV service. The compression arguments are loaded from TiKV config.
It will affect TiKV's response to TiDB.

Signed-off-by: ekexium <[email protected]>

Co-authored-by: ekexium <[email protected]>
ti-chi-bot bot pushed a commit that referenced this issue Jul 29, 2024
@ti-chi-bot @ekexium
grpc: set compression args for TiKV service (#17180) (#17312)
39b4eec 
close #17176

Set compression arguments for TiKV service. The compression arguments are loaded from TiKV config.
It will affect TiKV's response to TiDB.

Signed-off-by: ekexium <[email protected]>

Co-authored-by: ekexium <[email protected]>
RidRisR pushed a commit to RidRisR/tikv that referenced this issue Aug 5, 2024
@ti-chi-bot @ekexium
@ti-chi-bot @RidRisR
grpc: set compression args for TiKV service (tikv#17180) (tikv#17313)
5b52390 
close tikv#17176

Set compression arguments for TiKV service. The compression arguments are loaded from TiKV config.
It will affect TiKV's response to TiDB.

Signed-off-by: ekexium <[email protected]>

Co-authored-by: ekexium <[email protected]>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Signed-off-by: RidRisR <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affects-5.4 affects-6.1 affects-6.5 affects-7.1 affects-7.5 affects-8.1 report/customer Customers have encountered this bug. type/enhancement Type: Issue - Enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

grpc: set compression args for TiKV service ekexium/tikv
3 participants
@ekexium @seiya-annie @ti-chi-bot