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

Issue 1338 - Add reporter for DynamoDBQueryTracker #1382

Merged
merged 18 commits into from
Oct 3, 2023
Merged

Issue 1338 - Add reporter for DynamoDBQueryTracker #1382

merged 18 commits into from
Oct 3, 2023

Conversation

kr565370
Copy link
Collaborator

@kr565370 kr565370 commented Sep 28, 2023 •
edited
Loading

Make sure you have checked all steps below.

Issue

Tests

  • My PR adds the following tests OR does not need testing for this extremely good reason:
    • JsonQueryTrackerReportTest
    • StandardQueryTrackerReportTest
    • Manually tested against real instance

Documentation

  • In case of new functionality, my PR adds documentation that describes how to use it, or I have linked to a
    separate issue for that below.
  • If I have added, removed, or updated any external dependencies used in the project, I have updated the
    NOTICES file to reflect this.

@kr565370
Add ability to get all queries from DynamoDBQueryTracker
9e60679
@kr565370
Move job query tests into job package
4227ebd
@kr565370
Start on QueryTrackerReporter tests
9289b17
@kr565370
Add test for mixed queries
eeec096
@kr565370
Move test resources into query folder
21fc38e
@kr565370
Add test for subqueries and create test helpers for TrackedQuery
fdd8c16
@kr565370
Fix checkstyle issue
cbe73b4
@kr565370
Add query by state to DynamoDBQueryTracker
bd226ea
@kr565370
Start on test for checking queries which are queued
f7086d0
@kr565370
Add test for reporting queued queries
35692f5
@kr565370
Fix incorrect resource path
842d42a
@kr565370
Add test for reporting in progress queries
26080b4
@kr565370
Merge branch 'main' into 1338-add-script-to-report-on-query-tracker
bc32264
@kr565370
Add test for reporting completed and failed queries
c45515a
@kr565370
Add test for json reporter
7aa514d
@kr565370
Add script to run QueryTrackerReport
cc771ae
@kr565370
Adjust prompt messages and fix argument count
bb5561d
@kr565370
Include default subQueryId value in reports
32376ab
@kr565370 kr565370 marked this pull request as ready for review October 3, 2023 12:08
@gaffer01 gaffer01 removed their assignment Oct 3, 2023
@kr565370 kr565370 merged commit c3f077f into main Oct 3, 2023
@kr565370 kr565370 deleted the 1338-add-script-to-report-on-query-tracker branch October 3, 2023 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gaffer01 gaffer01 gaffer01 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add script to check status of queries in query tracker
2 participants
@kr565370 @gaffer01