Closed tomncooper closed 1 week ago
Thanks for the PR. Nicely written demo! Regarding the setup.sh script, I see a similar script in @SamBarker's PR #17. Should we perhaps come up with a common script for setting up everything, and starts up a different demo app based on the given argument or something?
@tinaselenge Good point about @SamBarker's script. I should have checked the open PRs! A common way to set-up the data generator would be a good thing. We will be adding more demos soon and it would be good not to re-invent the wheel.
@SamBarker is there much work needed to get your script merged. I am happy to help if you don't have time?
@SamBarker is there much work needed to get your script merged. I am happy to help if you don't have time?
Oh f*ck sorry. I think it needs the "upstream" branch fixed or removed (too add back in a separate PR). Feel free to push to that branch if you have the cycles
Thanks for the reviews folks. @SamBarker I am gonna merge this PR so we have the tutorial up. But we should def work to combine the data gen setup scripts and have a common setup for all demos.
This PR adds a simple tutorial for an ETL process using Flink SQL: