---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23705/#review48195
---
Thanks for the patch. A couple of comments below.
core/src/main/scala/kafka/tools/DumpLogSegments.scala
https://reviews.apache.org/r/23705/#comment84553
Shouldn't Decoder be kafka.serializer.Decoder?
Also, capitalize custom.
core/src/main/scala/kafka/tools/DumpLogSegments.scala
https://reviews.apache.org/r/23705/#comment84554
Could we add a decoder for the key too?
- Jun Rao
On July 19, 2014, 8:28 a.m., Manikumar Reddy O wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23705/
---
(Updated July 19, 2014, 8:28 a.m.)
Review request for kafka.
Bugs: KAFKA-1192
https://issues.apache.org/jira/browse/KAFKA-1192
Repository: kafka
Description
---
Support given for custom deserialization of messages
Diffs
-
core/src/main/scala/kafka/tools/DumpLogSegments.scala
6daf87b25a48a51aafb7dbe8d0c0371e0ea7501f
Diff: https://reviews.apache.org/r/23705/diff/
Testing
---
Thanks,
Manikumar Reddy O