Skip to content

Commit

Permalink
Add application_name to compute activity monitor connection string
Browse files Browse the repository at this point in the history
This was missed in my previous attempt to mark every connection string
with an application name. See 0c3e3a8.
  • Loading branch information
tristan957 committed Jun 27, 2024
1 parent 1d66ca7 commit 5700233
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion compute_tools/src/monitor.rs
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,11 @@ const MONITOR_CHECK_INTERVAL: Duration = Duration::from_millis(500);
// should be handled gracefully.
fn watch_compute_activity(compute: &ComputeNode) {
// Suppose that `connstr` doesn't change
let connstr = compute.connstr.as_str();
let mut connstr = compute.connstr.clone();
connstr
.query_pairs_mut()
.append_pair("application_name", "compute_activity_monitor");
let connstr = connstr.as_str();

// During startup and configuration we connect to every Postgres database,
// but we don't want to count this as some user activity. So wait until
Expand Down

1 comment on commit 5700233

@github-actions
Copy link

Choose a reason for hiding this comment

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

3028 tests run: 2902 passed, 0 failed, 126 skipped (full report)


Flaky tests (2)

Postgres 15

Code coverage* (full report)

  • functions: 32.7% (6917 of 21145 functions)
  • lines: 50.1% (54155 of 108105 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
5700233 at 2024-06-27T21:06:57.755Z :recycle:

Please sign in to comment.