Skip to content

Conversation

jacopocarlini
Copy link
Contributor

@jacopocarlini jacopocarlini commented Apr 17, 2025

List of Changes

✨ Added Python script to extract stand-in events within specified date range.

Motivation and Context

PAGOPA-2991

How Has This Been Tested?

manually

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

✨ Added Python script to extract stand-in events within specified date range.

PAGOPA-2991
@jacopocarlini jacopocarlini requested a review from a team as a code owner April 17, 2025 10:32
Copy link

The default action is to increase the PATCH number of SEMVER. Set IGNORE-FOR-RELEASE if you want to skip SEMVER bump. BREAKING-CHANGE and NEW-RELEASE must be run from GH Actions section manually.

@github-actions github-actions bot added the patch label Apr 17, 2025
Copy link

The default action is to increase the PATCH number of SEMVER. Set IGNORE-FOR-RELEASE if you want to skip SEMVER bump. BREAKING-CHANGE and NEW-RELEASE must be run from GH Actions section manually.

Copy link

This pull request does not contain a valid label. Please add one of the following labels: [patch, ignore-for-release]

Copy link
Contributor

@pasqualespica pasqualespica left a comment

Choose a reason for hiding this comment

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

LGTM evaluate to use read replica in PROD for this query

"""Create and return a Cosmos DB client using connection string"""
try:
connection_string = args.cosmos_connection_string
client = CosmosClient.from_connection_string(connection_string)
Copy link
Contributor

Choose a reason for hiding this comment

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

to avoid abuse RU in prod env , use preferred_locations to switch report query on read replica
https://learn.microsoft.com/en-us/azure/cosmos-db/nosql/best-practice-python#sdk-usage

example see here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thank you for this advice

Copy link

Comment this PR with update_code to format the code. Consider to use pre-commit to format the code.

Copy link
Contributor

@pasqualespica pasqualespica left a comment

Choose a reason for hiding this comment

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

🚀

@jacopocarlini jacopocarlini merged commit 0d75908 into main Apr 18, 2025
8 of 11 checks passed
@jacopocarlini jacopocarlini deleted the PAGOPA-2991-script-di-supporto-per-le-proiezioni branch April 18, 2025 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants