-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add documentation for streaming usecase #9070
Add documentation for streaming usecase #9070
Conversation
@Tangruilin I had some spare time and decided to work on this issue. Feel free to review this PR, and also add new examples to show capabilities of datafusion. |
Okey, thanks very much for help |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I added some clarifications
Co-authored-by: Mehmet Ozan Kabak <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @mustafasrepo 🙏 -- this looks great
I left some small suggestions on how to maybe improve it, but I don't think they are required to merge this PR
Co-authored-by: Andrew Lamb <[email protected]>
Love it -- thanks again @mustafasrepo |
Which issue does this PR close?
Improves the situation on epic:#9016
It closes first and second tickets
Rationale for this change
What changes are included in this PR?
Are these changes tested?
Yes
Are there any user-facing changes?