Skip to content

Commit

Permalink
Change in record formatter to add namespace to topic in ad-hoc tests …
Browse files Browse the repository at this point in the history
…(didn't work previously when naming strategy was set)
  • Loading branch information
Szymon Bogusz committed Nov 27, 2024
1 parent fae79e6 commit f7e4dd4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,7 @@ package pl.touk.nussknacker.engine.schemedkafka.schemaregistry
import cats.data.Validated
import io.confluent.kafka.schemaregistry.ParsedSchema
import pl.touk.nussknacker.engine.kafka.{KafkaConfig, UnspecializedTopicName}
import pl.touk.nussknacker.engine.schemedkafka.{
TopicSelectionStrategy,
TopicsMatchingPatternWithExistingSubjectsSelectionStrategy,
TopicsWithExistingSubjectSelectionStrategy
}
import pl.touk.nussknacker.engine.schemedkafka.TopicsWithExistingSubjectSelectionStrategy

trait SchemaRegistryClient extends Serializable {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import pl.touk.nussknacker.engine.api.process.TopicName
import pl.touk.nussknacker.engine.api.test.TestRecord
import pl.touk.nussknacker.engine.kafka.consumerrecord.SerializableConsumerRecord
import pl.touk.nussknacker.engine.kafka.{KafkaConfig, RecordFormatter, serialization}
import pl.touk.nussknacker.engine.schemedkafka.TopicsWithExistingSubjectSelectionStrategy
import pl.touk.nussknacker.engine.schemedkafka.schemaregistry.{
ContentTypes,
ContentTypesSchemas,
Expand Down Expand Up @@ -143,7 +142,7 @@ abstract class AbstractSchemaBasedRecordFormatter[K: ClassTag, V: ClassTag] exte

}

record.consumerRecord.toKafkaConsumerRecord(topic, serializeKeyValue)
record.consumerRecord.copy(topic = Some(topic.name)).toKafkaConsumerRecord(topic, serializeKeyValue)
}

protected def readRecordKeyMessage(
Expand Down

0 comments on commit f7e4dd4

Please sign in to comment.