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

KQL fixes #127

Open
wants to merge 6 commits into
base: microsoftfabric2.0
Choose a base branch
from

Conversation

alxdean
Copy link

@alxdean alxdean commented Mar 21, 2024

Small fixes to the kql scripts to make them run smoothly during a demo. table name needs to be in lowercase, as that's what created during setup. also, sql scripts have no place in kql querysets. if you want to keep them, I suggest to comment them out.

updated KQL table name to lowercase to match setup instructions
Removed sql and buggy kql from the script to make it run smoothly
@AllgeierSchweizAdmin
Copy link

@microsoft-github-policy-service agree

1 similar comment
@alxdean
Copy link
Author

alxdean commented Mar 21, 2024

@microsoft-github-policy-service agree

@alxdean
Copy link
Author

alxdean commented Mar 22, 2024

Just realised that my follow up committs also landed in this pull request. will work with cleaner branching next time. This pull request now includes documentation changes for streaming datasets and a bug fix where the realtime app kept failing to deploy properly. This pull request adds some clarity and stability to the use of realtime data to the project.

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.

2 participants