-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
422 issue nullpointerexception when starting a test with a key schema file serializer config #426
Conversation
… and other Serializers.
…ng-a-test-with-a-key-schema-file-serializer-config # Conflicts: # src/main/java/com/sngular/kloadgen/sampler/KafkaProducerSampler.java
…eSerializer to KeyFileSerializer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for collaborating with the project to help us improve!
This pull request hasn't been labeled as |
…ng-a-test-with-a-key-schema-file-serializer-config
…ng-a-test-with-a-key-schema-file-serializer-config
|
||
import org.apache.kafka.common.serialization.Serializer; | ||
|
||
public interface EnrichedRecordSerializer<T extends EnrichedRecord> extends Serializer<T> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no sense on that. Rethink.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a tag for those serializers that can accept a EnricherdRecord, instead of a GenericRecordSerializer. Is there a way to query this programatically?
@@ -0,0 +1,21 @@ | |||
package com.sngular.kloadgen.serializer; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think is is not needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand. Isn't the package
line always needed for classes?
No description provided.