kafka-tutorials icon indicating copy to clipboard operation
kafka-tutorials copied to clipboard

Suggestions to improve datacenter recipe

Open mjsax opened this issue 3 years ago • 4 comments

In https://confluentinc.github.io/ksqldb-recipes/real-time-analytics/datacenter/, we create overloaded_panels but it's not use anywhere? Wondering (without reading the text of the recipe) if it's intended as "final output", i.e., this recipe has two results instead of just one?

Also I think the last two queries could be merged.

mjsax avatar Dec 09 '21 23:12 mjsax

@mjsax thanks for the feedback. @rspurgeon can you take a look at this one?

ybyzek avatar Dec 10 '21 12:12 ybyzek

The point of overloaded_panels was to provide a 'final output' stream for a user to work with. I'll add some text about querying it as a use case.

There was some discussion (elsewhere) about merging ksqlDB commands into one command. I can see pros/cons for keeping streams / tables independent or combining them. Where did we land on this in other recipes?

rspurgeon avatar Jan 07 '22 18:01 rspurgeon

There was some discussion (elsewhere) about merging ksqlDB commands into one command. I can see pros/cons for keeping streams / tables independent or combining them. Where did we land on this in other recipes?

As you say, there are pros and cons to merging (pro: shorter; con: more confusing for newbies). Please merge (or not) at your discretion.

ybyzek avatar Jan 07 '22 18:01 ybyzek

In general, we recommend to merge queries, because running one persistent query is more efficient than running two. That is why it might be good to show efficient code in the recipes, too?

mjsax avatar Jan 08 '22 00:01 mjsax