----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15760/ -----------------------------------------------------------
Review request for samza. Repository: samza Description ------- Added %s tokens to a number of places in KafkaConfig where a random UUID should be added to an id string. Diffs ----- samza-kafka/src/main/scala/org/apache/samza/config/KafkaConfig.scala 4947b87e1a51654a661ddf3ab606478f0967067d samza-kafka/src/test/scala/org/apache/samza/config/TestKafkaConfig.scala PRE-CREATION Diff: https://reviews.apache.org/r/15760/diff/ Testing ------- Added TestKafkaConfig.scala: *Create 2 ConsumerConfig objects with no clientId or groupId arguments passed to constructor. Confirmed that the generated client and group ids are indeed different due to the included UUID. * Create a ConsumerConfig object with explicit values for clientId and GroupId passed to constructor. Confirmed the created object has the specified client and group ids. * Do similar validation with ProducerConfig objects Thanks, Garry Turkington
