Open jukkakarvanen opened 5 years ago
Similar problems noticed also in other classes.
Yeah, to be honest, my first implementation was "quick and dirty", also I am not coding daily anymore - thus, I forgot some best practices about Java coding in last years :-)
Thank you so much for these improvements and pull requests.
I merged your fix. Thank you.
I will keep the ticket open as reminder for the other classes. I will refactor the other classes, too.
@jukkakarvanen Hey! im a newbie at this... im trying to run this repo using mvn clean package. Do I have to use kafka as well? or just use mvn clean package and everything would run? kindly help me out on this.
Kafka_Streams_TensorFlow_Image_Recognition_Example use class static variables to pass information between foreach and mapValues steps.
So I expect rather interesting problems if executed with multiple threads.
I have fix for this, but it is on top of changes related to #11