Skip to content

server: remove stale comment#96768

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
andreimatei:small.remove-comment
Feb 8, 2023
Merged

server: remove stale comment#96768
craig[bot] merged 1 commit intocockroachdb:masterfrom
andreimatei:small.remove-comment

Conversation

@andreimatei
Copy link
Copy Markdown
Contributor

Release note: None
Epic: None

Release note: None
Epic: None
@andreimatei andreimatei requested review from a team as code owners February 8, 2023 01:29
@blathers-crl
Copy link
Copy Markdown

blathers-crl bot commented Feb 8, 2023

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@andreimatei
Copy link
Copy Markdown
Contributor Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Feb 8, 2023

Build succeeded:

@craig craig bot merged commit 72e3b1f into cockroachdb:master Feb 8, 2023
katmayb pushed a commit to katmayb/cockroach that referenced this pull request Feb 9, 2023
96768: server: remove stale comment r=andreimatei a=andreimatei

Release note: None
Epic: None

96781: rpc: improve detection of onlyOnceDialer redials r=erikgrinaker a=tbg

While looking into cockroachdb#96543, I wasn't 100% sure we weren't accidentally
redialing a connection internally. This improved logging and the test
makes it more obvious that things are working as intended.

Touches cockroachdb#96543.

Epic: none
Release note: None

96782: roachtest: verbose gRPC logging for admission-control/tpcc-olap r=irfansharif a=tbg

Closes cockroachdb#96543.

Next time we'll know more.

Epic: none
Release note: None

Co-authored-by: Andrei Matei <andrei@cockroachlabs.com>
Co-authored-by: Tobias Grieger <tobias.b.grieger@gmail.com>
@andreimatei andreimatei deleted the small.remove-comment branch February 22, 2023 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants