yyfhust commented on PR #34505: URL: https://github.com/apache/beam/pull/34505#issuecomment-2775661324
> Thanks! Would it be possible to extend the test coverage? I think you could modify a simpler test like testKafkaIOWriteWithErrorHandler in KafkaIOIT.java to have a custom deserializer (perhaps just a no-op subclass of StringSerializer for example) without a registered coder. That would help prevent regressions as well. Yes, will do. Actually I was waiting for someone to point me to the place where I can write unit/integration tests 🤣 . Thanks. 🙇♂️ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@beam.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
