Skip to content

feat(statsTabV2): bring stats tab v2 to OSS #13431

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

v-tarasevich-blitz-brain
Copy link
Contributor

No description provided.

@github-actions github-actions bot added product PR or Issue related to the DataHub UI/UX devops PR or Issue related to DataHub backend & deployment labels May 6, 2025
Copy link

codecov bot commented May 6, 2025

Codecov Report

Attention: Patch coverage is 41.51491% with 3629 lines in your changes missing coverage. Please review.

❌ Unsupported file format

Upload processing failed due to unsupported file format. Please review the parser error message:

Error parsing JUnit XML in /home/runner/work/datahub/datahub/metadata-io/build/test-results/test/TEST-com.linkedin.metadata.graph.search.elasticsearch.SearchGraphServiceElasticSearchTest.xml at 117:1058

Caused by:
    RuntimeError: Error converting computed name to ValidatedString
    
    Caused by:
        string is too long

For more help, visit our troubleshooting guide.

Files with missing lines Patch % Lines
...data/timeseries/elastic/OperationsServiceUtil.java 0.00% 209 Missing ⚠️
...2/graphs/ChangeHistoryGraph/ChangeHistoryGraph.tsx 14.28% 156 Missing ⚠️
.../Stats/StatsTabV2/columnStats/ColumnStatsTable.tsx 20.66% 119 Missing ⚠️
...components/ChangeHistoryDrawer/useGetOperations.ts 14.06% 110 Missing ⚠️
...geHistoryGraph/components/ChangeHistoryPopover.tsx 39.49% 95 Missing ⚠️
...sTabV2/graphs/QueryCountGraph/useQueryCountData.ts 24.19% 94 Missing ⚠️
...tatsTabV2/columnStats/useGetColumnStatsColumns.tsx 22.03% 92 Missing ⚠️
...s/Dataset/Stats/StatsTabV2/historical/TopUsers.tsx 31.62% 80 Missing ⚠️
...onents/ChangeHistoryDrawer/ChangeHistoryDrawer.tsx 38.75% 79 Missing ⚠️
...TabV2/graphs/StorageSizeGraph/StorageSizeGraph.tsx 23.23% 76 Missing ⚠️
... and 112 more

📢 Thoughts on this report? Let us know!

Copy link

alwaysmeticulous bot commented May 6, 2025

🔴 Meticulous spotted visual differences in 34 of 1318 screens tested: view and approve differences detected.

Meticulous evaluated ~8 hours of user flows against your PR.

Last updated for commit c519d7e. This comment will update as new commits are pushed.

@v-tarasevich-blitz-brain v-tarasevich-blitz-brain force-pushed the vt--bring-stats-tab-v2-to-oss branch from 692e278 to def4931 Compare May 7, 2025 14:25
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain force-pushed the vt--bring-stats-tab-v2-to-oss branch from def4931 to 72b2b20 Compare May 7, 2025 15:15
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain marked this pull request as ready for review May 7, 2025 16:02
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label May 7, 2025
@v-tarasevich-blitz-brain v-tarasevich-blitz-brain force-pushed the vt--bring-stats-tab-v2-to-oss branch from 72b2b20 to 22e1d50 Compare May 12, 2025 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops PR or Issue related to DataHub backend & deployment needs-review Label for PRs that need review from a maintainer. product PR or Issue related to the DataHub UI/UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants