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

[Fabric E2E Sample] | Add workspace name to lakehouse objects #1087

Merged
merged 13 commits into from
Feb 10, 2025

Conversation

sreedhar-guda
Copy link
Contributor

Type of PR

  • Code changes

Purpose

To have fully qualified names to the lakehouse objects and avoid relative references.

Author pre-publish checklist

  • No PII in logs
  • Made corresponding changes to the documentation

Issues Closed or Referenced

  • Closes #issue_number
  • References #issue_number

Updated notebooks to have fully qualified names when referring to lakehouse.
Updated lakehouse ddls to have workspace name to have fully qualified name when referencing lakehouse objects. Aldo modified the sqls to avoid creating the table if it exists.
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@naga-nandyala naga-nandyala changed the title Add workspace name to lakehouse objects [Fabric E2E Sample] | Add workspace name to lakehouse objects Feb 10, 2025
Copy link
Contributor

@naga-nandyala naga-nandyala left a comment

Choose a reason for hiding this comment

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

looks good to me

@promisinganuj promisinganuj merged commit 81d626c into feat/e2e-fabric-dataops-sample-v0-2 Feb 10, 2025
3 checks passed
@sreedhar-guda sreedhar-guda deleted the sguda/add-fqns branch February 10, 2025 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants