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.

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

Issue-12914 -- Split profiler, data quality and data insight table from entity_time_series_extension #12991

Merged
merged 18 commits into from
Aug 28, 2023

Conversation

TeddyCr
Copy link
Contributor

@TeddyCr TeddyCr commented Aug 24, 2023

Describe your changes:

Fixes #12914

Split profiler, data quality and data insight table from entity_time_series_extension

Type of change:

  • Bug fix
  • Improvement
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Checklist:

  • I have read the CONTRIBUTING document.
  • My PR title is Fixes <issue-number>: <short explanation>
  • I have commented on my code, particularly in hard-to-understand areas.
  • For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@TeddyCr TeddyCr requested a review from a team as a code owner August 24, 2023 10:10
@TeddyCr TeddyCr temporarily deployed to test August 24, 2023 10:11 — with GitHub Actions Inactive
@TeddyCr TeddyCr temporarily deployed to test August 24, 2023 10:11 — with GitHub Actions Inactive
@TeddyCr TeddyCr temporarily deployed to test August 24, 2023 10:11 — with GitHub Actions Inactive
@github-actions github-actions bot added Ingestion safe to test Add this label to run secure Github workflows on PRs labels Aug 24, 2023
@cypress
Copy link

cypress bot commented Aug 24, 2023

1 failed tests on run #28533 ↗︎

1 327 37 0 Flakiness 0

Details:

Merge branch 'main' into issue-12914
Project: openmetadata Commit: 0ad24b62f0
Status: Failed Duration: 28:43 💡
Started: Aug 25, 2023 7:48 AM Ended: Aug 25, 2023 8:17 AM
Failed  cypress/e2e/AddNewService/postgres.spec.js • 1 failed test • Tests with database postgresql

View Output Video

Test Artifacts
Postgres Ingestion > Verify if usage is ingested properly Output Screenshots Video

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@TeddyCr TeddyCr temporarily deployed to test August 24, 2023 11:54 — with GitHub Actions Inactive
@TeddyCr TeddyCr temporarily deployed to test August 25, 2023 07:26 — with GitHub Actions Inactive
@TeddyCr TeddyCr temporarily deployed to test August 25, 2023 07:26 — with GitHub Actions Inactive
@TeddyCr TeddyCr temporarily deployed to test August 25, 2023 07:26 — with GitHub Actions Inactive
@sonarqubecloud
Copy link

[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@TeddyCr TeddyCr merged commit 867e1c4 into open-metadata:main Aug 28, 2023
TeddyCr added a commit that referenced this pull request Sep 6, 2023
…om `entity_time_series_extension` (#12991)

* fix: write testCase result testSuite to compute summary

* fix: empty commit to trigger CI

* fix: extracted report data out of entity_extension_time_series and into report_data_time_series

* fix: split profiler data  from entity_extension_time_series

* fix: split data quality data time series table

* fix: updated migration file

(cherry picked from commit 867e1c4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Ingestion safe to test Add this label to run secure Github workflows on PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Split Profiler and DQ data into their own tables
2 participants