[GitHub] [kafka] pjmagee commented on pull request #11442: KAFKA-7883 add schema.namespace support to SetSchemaMetadata SMT in Kafka Connect

2022-08-22 Thread GitBox
pjmagee commented on PR #11442: URL: https://github.com/apache/kafka/pull/11442#issuecomment-1222517478 Hi @hachikuji Sorry to pull you in on this PR - I notice you have been active on some merged PR's quite recently. We have had some input from others regarding the process around creating

[GitHub] [kafka] pjmagee commented on pull request #11442: KAFKA-7883 add schema.namespace support to SetSchemaMetadata SMT in Kafka Connect

2022-08-10 Thread GitBox
pjmagee commented on PR #11442: URL: https://github.com/apache/kafka/pull/11442#issuecomment-1210700120 @mimaison Hey, It looks like @mnegodaev has followed all the right procedures and we have the KIP. As someone with more experience on this repo, what are our next steps? -- This is a

[GitHub] [kafka] pjmagee commented on pull request #11442: KAFKA-7883 add schema.namespace support to SetSchemaMetadata SMT in Kafka Connect

2022-07-12 Thread GitBox
pjmagee commented on PR #11442: URL: https://github.com/apache/kafka/pull/11442#issuecomment-1181487955 @mnegodaev It looks like this is something you would add to this PR a link to a KIP. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [kafka] pjmagee commented on pull request #11442: KAFKA-7883 add schema.namespace support to SetSchemaMetadata SMT in Kafka Connect

2022-07-07 Thread GitBox
pjmagee commented on PR #11442: URL: https://github.com/apache/kafka/pull/11442#issuecomment-1178051631 @mjsax I can see you are recently doing some things, is there any way we can get @mnegodaev PR merged? -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [kafka] pjmagee commented on pull request #11442: KAFKA-7883 add schema.namespace support to SetSchemaMetadata SMT in Kafka Connect

2022-07-05 Thread GitBox
pjmagee commented on PR #11442: URL: https://github.com/apache/kafka/pull/11442#issuecomment-1175180448 @rhauch @mnegodaev this is looking pretty good now. Would others need to have reviewed this before being merged in? I'm not too clued in on the open source contributions here but it look

[GitHub] [kafka] pjmagee commented on pull request #11442: KAFKA-7883 add schema.namespace support to SetSchemaMetadata SMT in Kafka Connect

2022-06-17 Thread GitBox
pjmagee commented on PR #11442: URL: https://github.com/apache/kafka/pull/11442#issuecomment-1158681036 Would I be correct with the following example too? ``` { "transforms" : "TransformSchema", "transforms.TransformSchema.type" : "org.apache.kafka.connect.transforms.SetS

[GitHub] [kafka] pjmagee commented on pull request #11442: KAFKA-7883 add schema.namespace support to SetSchemaMetadata SMT in Kafka Connect

2022-06-16 Thread GitBox
pjmagee commented on PR #11442: URL: https://github.com/apache/kafka/pull/11442#issuecomment-1157733922 For context, we have had to go from 2 connectors to a huge number of connectors as a workaround, which has been a crazy amount of configuration for us, based on how we currently use k

[GitHub] [kafka] pjmagee commented on pull request #11442: KAFKA-7883 add schema.namespace support to SetSchemaMetadata SMT in Kafka Connect

2022-06-16 Thread GitBox
pjmagee commented on PR #11442: URL: https://github.com/apache/kafka/pull/11442#issuecomment-1157553640 I really want this! This is exactly what our team needs to resolve issues with Avro namespaces. -- This is an automated message from the Apache Git Service. To respond to the message, p