Stats with Opentelemetry #141
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Also move superset_dashboard in /dff/config/
Since superset_dashboard was moved to dff/config need to update its location in sourcefiles Also add superset_dashboard to MANIFEST.in
Pipeline's interface was changed between versions v1 and v2. This commit changes imported names in stats examples to reflect that. Note: Also mass replace showed that documentation for pipeline is outdated.
Since documentation building requires all files in examples dir to be named differently, rename _utils.
- Add dot_path_to_addon to find examples location - Explicitly list examples Signed-off-by: Roman Zlobin <[email protected]>
- Add .dockerignore from stats - Add volumes to existing services - Add [.env_file] to new services - Add CLICKHOUSE config to .env_file - Add df_stats/dockerfile as dockerfile_stats Signed-off-by: Roman Zlobin <[email protected]>
- Add sql_dependencies (a lot of dbs use sqlalchemy so I decided to move it to a separate list) - Add asyncpg as postgres dependency (stats used it instead of psycopg) - Add an entry point for dff.stats Signed-off-by: Roman Zlobin <[email protected]>
This commit does not rename table names from df_stats to dff.stats
This change moves pings to dbs from tests/db_connector/test_dbs.py to tests/db_list.py
This reverts commit 58dcc97.
pseusys
reviewed
Aug 21, 2023
Comment on lines
+113
to
+119
| """ | ||
| Type of object, that is passed to components in runtime. | ||
| Contains current component info (`name`, `path`, `timeout`, `asynchronous`). | ||
| Also contains `execution_state` - a dictionary, | ||
| containing execution states of other components mapped to their paths. | ||
| """ | ||
|
|
There was a problem hiding this comment.
So maybe we should move it to another one?
…archiving with shutil module function
Correct the workflow for dashboard building; update and improve user_guide; Co-authored-by: Roman Zlobin <[email protected]>
RLKRo
reviewed
Aug 23, 2023
RLKRo
requested changes
Aug 24, 2023
RLKRo
approved these changes
Aug 25, 2023
…mework into feat/otlponly
…mework into feat/otlponly
kudep
approved these changes
Sep 5, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Collect and export telemetry information using Opentelemetry Python sdk, Clickhouse, and Apache Superset.
Checklist