feat(ingestion): add ability to exclude archived mode reports.#2
feat(ingestion): add ability to exclude archived mode reports.#2NehaGslab wants to merge 37 commits into
Conversation
Co-authored-by: Aseem Bansal <asmbansal2@gmail.com>
|
Hello @NehaGslab 😄 Thank you so much for opening a pull request!
|
…b-project#14875) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…b-project#14950) Co-authored-by: acryl-hyejin <130044278+acryl-hyejin@users.noreply.github.com> Co-authored-by: Hyejin Yoon <0327jane@gmail.com>
| reports_page = [ | ||
| report | ||
| for report in reports_page | ||
| if report["archived"] is False |
There was a problem hiding this comment.
it would be good idea to keep num_archived_excluded or so counter for observability.
… invite token (datahub-project#14941) Co-authored-by: Deepak Garg <deepak.garg@datahub.com>
Co-authored-by: trialiya <trialiya@gmail.com> Co-authored-by: Deepak Garg <deepak.garg@datahub.com>
Addressed review comments Co-authored-by: Tamas Nemeth <treff7es@gmail.com>
…he table is deleted. (datahub-project#14763)
Co-authored-by: Deepak Garg <deepak.garg@datahub.com>
Adds two tasks reload and reloadEnv which work with any debug quickstart profile. They automatically detect the running profile to restart the correct set of services based on the running profile. The earlier profile specific tasks DebugReload[Env] are no longer needed (but still present, they now just delegate the call to these new tasks. This also now handles ReloadEnv correctly for services that support hot-reloading. Some of the python service container names were not consistent across profiles, so this now detects the names based on the prefix as part of the auto-detection of running profile and services.
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
🚫 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2004:style:24:13:
There was a problem hiding this comment.
📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2086:info:26:10: Double quote to prevent globbing and word splitting [shellcheck]
There was a problem hiding this comment.
📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2086:info:56:20: Double quote to prevent globbing and word splitting [shellcheck]
16 participants
No description provided.