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

feat(ingest): powerbi # set dataset_type_mapping to all supported data platform #7598

Conversation

siddiquebagwan
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Mar 15, 2023
Oracle: oracle
Sql: mssql
GoogleBigQuery: bigquery
PostgreSql:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The whole purpose is to make the user do LESS. Why would we include this in our sample recipe? Please reove the entire field from our starter recipe

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This to give example for how to set the platform_instance with recipe, I will remove it in next PR when we make this field deprecated

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is for reference for how to set platform_instance. We can remove this field once we deprecated it in next PR.

@@ -651,7 +656,7 @@ def test_powerbi_ingest_urn_lower_case(
},
}
)

pipeline.config
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deleted

@vercel
Copy link

vercel bot commented Mar 21, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
docs-website ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 21, 2023 at 7:16AM (UTC)

@vercel vercel bot temporarily deployed to Preview March 21, 2023 07:16 Inactive
@siddiquebagwan
Copy link
Contributor Author

PR: #7632

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ingestion PR or Issue related to the ingestion of metadata
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants