Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

Simplify Cassandra server in thread names #6092

Merged
merged 2 commits into from
Jun 17, 2022
Merged

Simplify Cassandra server in thread names #6092

merged 2 commits into from
Jun 17, 2022

Conversation

schlosna
Copy link
Contributor

@schlosna schlosna commented Jun 17, 2022

Goals (and why):
Avoid including set of Cassandra proxy addresses in thread names as seen in example:

Atlas getRowsColumnRange 1 rows from namespace.table on CassandraServer{cassandraHostName=cassandra-123.foo.bar, reachableProxyIps=[cassandra-123.foo.bar/127.0.0.1:9160]} worker task-1234 calling cassandra host cassandra-123.foo.bar/127.0.0.1:9160 started at 2022-06-17T12:00:00.123456789Z - 1234567890

Additional follow on to #6076

==COMMIT_MSG==
Simplify Cassandra server in thread names

Avoid including set of Cassandra proxy addresses in thread names
==COMMIT_MSG==

Implementation Description (bullets):
Include only CassandraServer hostname and remove proxy IPs from generated ImmutableCassandraServer#toString()

Testing (What was existing testing like? What have you done to improve it?):

Concerns (what feedback would you like?):

Where should we start reviewing?:

Priority (whenever / two weeks / yesterday):

Avoid including set of Cassandra proxy addresses in thread names
@changelog-app
Copy link

changelog-app bot commented Jun 17, 2022

Generate changelog in changelog/@unreleased

Type

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

Simplify Cassandra server in thread names

Avoid including set of Cassandra proxy addresses in thread names

Check the box to generate changelog(s)

  • Generate changelog entry

Copy link
Contributor

@jeremyk-91 jeremyk-91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 thanks!

@bulldozer-bot bulldozer-bot bot merged commit 05a522a into develop Jun 17, 2022
@bulldozer-bot bulldozer-bot bot deleted the ds/tweaks branch June 17, 2022 20:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants