Skip to content

Add "count distinct" option to CloudWatch query script#8887

Merged
aduth merged 6 commits intomainfrom
aduth-count-distinct-stitch
Jul 31, 2023
Merged

Add "count distinct" option to CloudWatch query script#8887
aduth merged 6 commits intomainfrom
aduth-count-distinct-stitch

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Jul 27, 2023

🎫 Ticket

Supports LG-7708

🛠 Summary of changes

Adds new --count-distinct option to bin/query-cloudwatch to help produce a total distinct count from a query that may need to be stitched together with the --complete functionality.

📜 Testing Plan

Example usage:

bin/query-cloudwatch --from 1h --env int --app idp --log events.log --count-distinct properties.user_id --query "filter name = 'Multi-Factor Authentication' and properties.event_properties.success = 1"

changelog: Internal, Log Querying, Add count distinct option to log querying script
aduth and others added 4 commits July 28, 2023 08:31
See: #8887 (comment)

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
@aduth aduth marked this pull request as ready for review July 28, 2023 12:50
@aduth aduth changed the title Add "count distinct" option to CloudWatch query script LG-7708: Add "count distinct" option to CloudWatch query script Jul 28, 2023
@aduth aduth changed the title LG-7708: Add "count distinct" option to CloudWatch query script Add "count distinct" option to CloudWatch query script Jul 28, 2023
@aduth aduth requested a review from zachmargolis July 28, 2023 12:51
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM one small comment

aduth added a commit that referenced this pull request Jul 31, 2023
See: #8887 (comment)

Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
@aduth aduth merged commit 561a281 into main Jul 31, 2023
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.

2 participants