-
Notifications
You must be signed in to change notification settings - Fork 20
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.
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
[Framework] Caching port app config #628
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌊
|
||
|
||
class PortAppConfigCache: | ||
_port_app_config: PortAppConfig |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
_port_app_config: PortAppConfig | |
_port_app_config: PortAppConfig | None |
CHANGELOG.md
Outdated
|
||
### Improvements | ||
|
||
- Added caching to port-app-config.yml retrieval from port api |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
specify that itll only be used for live events
|
||
### Improvements | ||
|
||
- Added caching to port-app-config.yml retrieval from port api (only for live events) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Added caching to port-app-config.yml retrieval from port api (only for live events) | |
- Added caching to port-app-config.yml retrieval from port api (Applied only for non resync integration events like live events) |
Description
What - Caching port app config
Why - To reduce requests to port-api
How - Save the config in a class attribute and update if previous config request was more than a minute ago
Saves average of 1 second in response time for real time events when overloaded
Type of change
Please leave one option from the following and delete the rest:
Screenshots
Include screenshots from your environment showing how the resources of the integration will look.
API Documentation
Provide links to the API documentation used for this integration.