Github user tzulitai commented on a diff in the pull request:
https://github.com/apache/flink/pull/5958#discussion_r186337301
--- Diff:
flink-connectors/flink-connector-kafka-0.10/src/main/java/org/apache/flink/streaming/connectors/kafka/internal/Kafka010Fetcher.java
---
@@ -78,6 +79,69 @@ public Kafka010Fetcher(
useMetrics);
}
+ private class KafkaConsumerRecordWrapper10 implements
ConsumerRecordMetaInfo {
+ private static final long serialVersionUID =
2651665280744549935L;
+
+ private final ConsumerRecord<byte[], byte[]> consumerRecord;
+
+ public KafkaConsumerRecordWrapper10(ConsumerRecord<byte[],
byte[]> consumerRecord) {
+ this.consumerRecord = consumerRecord;
+ }
+
+ @Override
+ public byte[] getKey() {
+ return consumerRecord.key();
+ }
+
+ @Override
+ public byte[] getMessage() {
+ return consumerRecord.value();
+ }
+
+ @Override
+ public String getTopic() {
+ return consumerRecord.topic();
+ }
+
+ @Override
+ public int getPartition() {
+ return consumerRecord.partition();
+ }
+
+ @Override
+ public long getOffset() {
+ return consumerRecord.offset();
+ }
+
+ @Override
+ public long getTimestamp() {
+ return Long.MIN_VALUE;
--- End diff --
Doesn't Kafka 0.10 support record timestamps?
---
