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

Add table gcp_sql_database_instance_metric_connections_hourly closes #265 #268

Merged
merged 6 commits into from
Jul 8, 2021
Merged

Add table gcp_sql_database_instance_metric_connections_hourly closes #265 #268

merged 6 commits into from
Jul 8, 2021

Conversation

ParthaI
Copy link
Contributor

@ParthaI ParthaI commented Jun 29, 2021

Integration test logs

Logs 
Add passing integration test logs here

Example query results

Results

Basic info

select
  instance_id,
  timestamp,
  minimum,
  maximum,
  average,
  sample_count
from
  gcp_sql_database_instance_connections_hourly
order by
  timestamp;
+-----------------------+----------------------+---------+---------+-------------------+--------------+
| instance_id           | timestamp            | minimum | maximum | average           | sample_count |
+-----------------------+----------------------+---------+---------+-------------------+--------------+
| parker-aaa:db-connect | 2021-06-28T11:24:00Z | 8       | 9       | 8.976744186046512 | 43           |
| parker-aaa:db-connect | 2021-06-28T12:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T13:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T14:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T15:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T16:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T17:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T18:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T19:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T20:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T21:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T22:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-28T23:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-29T00:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-29T01:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-29T02:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-29T03:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-29T04:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-29T05:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-29T06:24:00Z | 9       | 9       | 9                 | 60           |
| parker-aaa:db-connect | 2021-06-29T07:24:00Z | 9       | 9       | 9                 | 60           |
+-----------------------+----------------------+---------+---------+-------------------+--------------+

Intervals averaging fewer than 9 connections

select
  instance_id,
  timestamp,
  round(minimum::numeric,2) as min_connection,
  round(maximum::numeric,2) as max_connection,
  round(average::numeric,2) as avg_connection,
  sample_count
from
  gcp_sql_database_instance_connections_hourly
where average < 9
order by timestamp;
+-----------------------+----------------------+----------------+----------------+----------------+--------------+
| instance_id           | timestamp            | min_connection | max_connection | avg_connection | sample_count |
+-----------------------+----------------------+----------------+----------------+----------------+--------------+
| parker-aaa:db-connect | 2021-06-28T11:28:00Z | 8.00           | 9.00           | 8.98           | 47           |
+-----------------------+----------------------+----------------+----------------+----------------+--------------+

@ParthaI ParthaI requested a review from Subhajit97 June 29, 2021 07:31
@ParthaI ParthaI self-assigned this Jun 29, 2021
@ParthaI ParthaI linked an issue Jun 29, 2021 that may be closed by this pull request
Add table gcp_sql_database_instance_connections_hourly #265
Closed
@Subhajit97 Subhajit97 requested a review from rajlearner17 June 29, 2021 13:10
rajlearner17
rajlearner17 reviewed Jun 29, 2021
View reviewed changes
docs/tables/gcp_sql_database_instance_metric_connections_hourly.md
@@ -0,0 +1,55 @@
# Table: gcp_sql_database_instance_metric_connections_hourly

Copy link

Choose a reason for hiding this comment

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

should it be gcp_sql_database_instance_connections_hourly

Copy link
Contributor

Choose a reason for hiding this comment

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

@rajmohanty17 We have followed same naming pattern defined in AWS

rajlearner17
rajlearner17 requested changes Jun 29, 2021
View reviewed changes
Copy link

@rajlearner17 rajlearner17 left a comment

Choose a reason for hiding this comment

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

Validate the table names in the doc.

@Subhajit97 Subhajit97 requested a review from rajlearner17 June 30, 2021 14:07
rajlearner17
rajlearner17 approved these changes Jul 6, 2021
View reviewed changes
Copy link

@rajlearner17 rajlearner17 left a comment

Choose a reason for hiding this comment

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

LGTM

@Subhajit97 Subhajit97 requested a review from cbruno10 July 6, 2021 15:11
@cbruno10 cbruno10 changed the title Add table gcp_sql_database_instance_connections_hourly closes #265 Add table gcp_sql_database_instance_metric_connections_hourly closes #265 Jul 8, 2021
@cbruno10 cbruno10 merged commit 1368ee5 into main Jul 8, 2021
@cbruno10 cbruno10 deleted the issue-265 branch July 8, 2021 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Subhajit97 Subhajit97 Subhajit97 approved these changes

@cbruno10 cbruno10 cbruno10 approved these changes

@rajlearner17 rajlearner17 rajlearner17 approved these changes

Assignees

@ParthaI ParthaI

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add table gcp_sql_database_instance_connections_hourly
4 participants
@ParthaI @cbruno10 @rajlearner17 @Subhajit97