-
Notifications
You must be signed in to change notification settings - Fork 322
Increase KafkaIO version to 0.2.0 #504
Increase KafkaIO version to 0.2.0 #504
Conversation
Recent PR GoogleCloudPlatform#491 changes how KafkaIO splits. This makes it incompatible with Dataflow update across these two versions.
+R: @dhalperi |
Suggest creating a README.md |
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.
Please add README.md per our discussion.
Added README.md. |
```java | ||
PCollection<KV<Long, String>> kafkaRecords = | ||
pipeline | ||
.applY(KafkaIO.read() |
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.
lowercase y
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.
Done
.withBootstrapServers("broker_1:9092,broker_2:9092") | ||
.withTopic("results") | ||
.withKeyCoder(BigEndianLongCoder.of()) | ||
.withValueCoder(StringUtf8Coder.of()) |
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.
missing last line?
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.
Done.
# KafkaIO : Dataflow Unbounded Source and Sink for Kafka Topics | ||
|
||
KafkaIO provides unbounded sources and sinks for [Kafka](https://www.firebase.com/) | ||
topics. Kafka version 0.9 and above are supported. |
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.
is this correct, or are code changes needed to support Kafka 0.10 or 0.11?
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.
Works with 0.10, but haven't tested with 0.11 (kafka master?). We can remove the line or mention specific versions tested.
+R: @dhalperi . I had addressed the comments immediately. I am not sure how to mark 'Changes Requested' as 'Done'. |
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.
Fixed.
Recent PR #491 changes how KafkaIO splits. This makes it incompatible
with Dataflow update across these two versions.