eolivelli commented on a change in pull request #9481:
URL: https://github.com/apache/pulsar/pull/9481#discussion_r572242740
##########
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/schema/generic/GenericAvroWriter.java
##########
@@ -52,4 +67,29 @@ public GenericAvroWriter(Schema schema) {
this.byteArrayOutputStream.reset();
}
}
+
+ /**
+ * This is an adapter from Pulsar GenericRecord to Avro classes.
+ */
+ private class GenericRecordAdapter extends SpecificRecordBase {
+ private GenericRecord message;
+
+ void setCurrentMessage(GenericRecord message) {
+ this.message = message;
+ }
+ @Override
+ public Schema getSchema() {
+ return schema;
+ }
+
+ @Override
+ public Object get(int field) {
+ return message.getField(schema.getFields().get(field).name());
+ }
+
+ @Override
+ public void put(int field, Object value) {
+ throw new UnsupportedOperationException();
+ }
+ }
Review comment:
Here we are in a writer and a writer is supposed to only read from the
object and not to alter the contents.
So it is good here that we prevent any change.
I cannot understand your point.
The integration tests makes an example of a Source that is pushing
GenericRecord instances and then those messages can be consumed by a standard
Pulsar consumer with AUTO_CONSUME
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
