Skip to content

Conversation

askumar27
Copy link
Contributor

  • Introduced Google Sheets as a new data platform with associated constants and logo.
  • Implemented Fivetran API client for handling Google Sheets connections.
  • Enhanced Fivetran source to emit dataset entities for Google Sheets connectors.
  • Added response models for Fivetran connection details specific to Google Sheets.
  • Updated configuration to include Google Sheets in the data platform mappings.

@github-actions github-actions bot added ingestion PR or Issue related to the ingestion of metadata product PR or Issue related to the DataHub UI/UX devops PR or Issue related to DataHub backend & deployment labels Oct 15, 2025
Copy link

codecov bot commented Oct 15, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
8246 1 8245 73
View the top 1 failed test(s) by shortest run time
dataset health test cypress/e2e/mutations/dataset_health.js::cypress/e2e/mutations/dataset_health.js
Stack Traces | 20.4s run time
2025-10-16T06:46:10.351Z
Timed out retrying after 10000ms: Expected to find content: 'Assertions 1 of 1 assertions are failing' but never did.

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Copy link

alwaysmeticulous bot commented Oct 15, 2025

✅ Meticulous spotted 0 visual differences across 958 screens tested: view results.

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

Expected differences? Click here. Last updated for commit f25ba17. This comment will update as new commits are pushed.

Copy link

codecov bot commented Oct 15, 2025

Bundle Report

Changes will increase total bundle size by 616 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 28.59MB 616 bytes (0.0%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: datahub-react-web-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-*.js 616 bytes 18.93MB 0.0%

Files in assets/index-*.js:

  • ./src/app/ingest/source/builder/constants.ts → Total Size: 6.76kB

  • ./src/images/google-sheets-logo.png → Total Size: 574 bytes

- Introduced Google Sheets as a new data platform with associated constants and logo.
- Implemented Fivetran API client for handling Google Sheets connections.
- Enhanced Fivetran source to emit dataset entities for Google Sheets connectors.
- Added response models for Fivetran connection details specific to Google Sheets.
- Updated configuration to include Google Sheets in the data platform mappings.
- Introduced a new subtype for Google Sheets named ranges in DatasetSubTypes.
- Enhanced Fivetran source to yield separate dataset entities for Google Sheets and their named ranges.
- Updated dataset properties to include relevant connection details and lineage information.
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 ingestion PR or Issue related to the ingestion of metadata product PR or Issue related to the DataHub UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant