-
Notifications
You must be signed in to change notification settings - Fork 90
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
Mistake in PRINT statement on stream-stream join tutorial #714
Comments
Is this statement not valid? |
I see -- I'm working in Confluent Cloud where the topic is called You're right, we lose the intentioned purpose, which is to show that the data is in a kafka topic and consumable by any consumer, not just ksqlDB. The safe thing to do would be to set the topic name explicitly with a |
cc: @bbejeck regarding @chuck-confluent 's comment above -- this an example of one issue KT would have to take into consideration if it presents Confluent Cloud workflows but validates locally. |
On a strange related note, I have never gotten this to work in confluent cloud for this tutorial:
I know there are messages in the topic because I was able to see them with a SELECT statement on the kstream. I can't print the topic, and when I go to the topic viewer and jump to offset 0 of each partition, I see nothing. I can print and inspect the messages of other topics, but something weird might be happening if there are very few messages in a topic. This might affect other tutorials in confluent cloud that don't use datagen, but instead use |
IMO this isn't a bug; it's working as expected locally. We definitely need to take into consideration running the tutorials in the cloud vs. locally. But it's important to realize that KT was set up entirely thinking of running locally. |
In this tutorial, there is a mistake in this PRINT statement:
The PRINT statement takes a topic as input, not a kstream. To use PRINT, the user would have to use the name of the derived topic, or an earlier step would have to use a
WITH kafka_topic = 'blah'
to set the topic name.A better approach would probably be to use this instead:
The text was updated successfully, but these errors were encountered: