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.

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

[DocDB] Ensure connection shutdown is only performed once #18807

Open
1 task done
karan-yb opened this issue Aug 22, 2023 · 0 comments
Open
1 task done

[DocDB] Ensure connection shutdown is only performed once #18807

karan-yb opened this issue Aug 22, 2023 · 0 comments
Labels
area/docdb YugabyteDB core features kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue

Comments

@karan-yb
Copy link
Contributor

karan-yb commented Aug 22, 2023

Jira Link: DB-7687

Description

As part of #18744 fix (https://phorge.dev.yugabyte.com/D27859), we added a change to ensure that we are only queuing the DestroyConnection once per connection. Figure out why we didn't run into this issue in prod, and simply the connection shutdown path by closing the connection inline.

I0818 10:07:31.725487 1868984320 connection.cc:134] 0x105a89d58 - Shutdown called with status Service unavailable (yb/rpc/reactor.cc:106): Shutdown connection (system error 58), existing Network error (yb/util/net/socket.cc:540): recvmsg error: Connection reset by peer (system error 54)
I0818 10:07:31.725772 1864396800 connection.cc:167] Connection (0x000000010758e7f8) server 127.0.0.1:59872 => 127.0.0.1:59864: Write failed: Network error (yb/util/net/socket.cc:447): sendmsg error: Broken pipe (system error 32)
I0818 10:07:31.727180 1868984320 connection.cc:167] Connection (0x0000000105a88fd8) client 127.0.0.1:59866 => 127.0.0.1:59864: Write failed: Network error (yb/util/net/socket.cc:447): sendmsg error: Broken pipe (system error 32)
I0818 10:07:31.727444 1868984320 connection.cc:134] 0x105a89d58 - Shutdown called with status Network error (yb/util/net/socket.cc:447): sendmsg error: Broken pipe (system error 32), existing Service unavailable (yb/rpc/reactor.cc:106): Shutdown connection (system error 58)
W0818 10:07:31.727458 1868984320 reactor.cc:758] Client_R000: Looking for Connection (0x0000000105a89d58) client 127.0.0.1:59871 => 127.0.0.1:59864

Warning: Please confirm that this issue does not contain any sensitive information

  • I confirm this issue does not contain any sensitive information.
@karan-yb karan-yb added area/docdb YugabyteDB core features status/awaiting-triage Issue awaiting triage labels Aug 22, 2023
@yugabyte-ci yugabyte-ci added kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue labels Aug 22, 2023
@yugabyte-ci yugabyte-ci added status/awaiting-triage Issue awaiting triage and removed status/awaiting-triage Issue awaiting triage labels Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docdb YugabyteDB core features kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue
Projects
None yet
Development

No branches or pull requests

2 participants