fix: test case result reindex performance - Githubissues

open-metadata / OpenMetadata

OpenMetadata is a unified metadata platform for data discovery, data observability, and data governance powered by a central metadata repository, in-depth column level lineage, and seamless team collaboration.

https://open-metadata.org

Apache License 2.0

5.59k stars 1.05k forks source link

fix: test case result reindex performance #18723

Closed TeddyCr closed 8 hours ago

TeddyCr commented 13 hours ago

Describe your changes:

remove redundant test case results list db calls
only fetch test suite result summary if summary field is present
pass explicit field when denormalizing test case result relationship on indexation

#

Type of change:

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

#

Checklist:

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

Bug fix

[x] I have added a test that covers the exact scenario we are fixing. For complex issues, comment the issue number in the test for future reference.

Improvement

[x] I have added tests around the new logic.
[x] For connector/ingestion changes: I updated the documentation.

sonarcloud[bot] commented 9 hours ago

Quality Gate passed for 'open-metadata-ingestion'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud