-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27391/
-----------------------------------------------------------
(Updated March 26, 2015, 7:17 p.m.)
Review request for kafka.
Bugs: KAFKA-1634
https://issues.apache.org/jira/browse/KAFKA-1634
Repository: kafka
Description (updated)
-------
1. Always override commit time to now, only change expire time accorinding to
the commit time.
2. Override expire time upon loading offsets of version 1.
3. Change new consumer to use the new version 2.
Diffs
-----
clients/src/main/java/org/apache/kafka/clients/consumer/internals/Coordinator.java
436f9b2a843bc8c44d17403f5880b6736a5d56a8
clients/src/main/java/org/apache/kafka/common/protocol/Protocol.java
101f382170ad6740b3f8ff2d27b93a64874a857f
clients/src/main/java/org/apache/kafka/common/protocol/types/Struct.java
7672a3a0d674d101078651956d7122059e59e6d5
clients/src/main/java/org/apache/kafka/common/requests/OffsetCommitRequest.java
94e9d376235b3288836807d8e8d2547b3743aad5
clients/src/test/java/org/apache/kafka/common/requests/RequestResponseTest.java
13237fd72da5448a3d596b882fef141f336f827d
core/src/main/scala/kafka/api/OffsetCommitRequest.scala
050615c72efe7dbaa4634f53943bd73273d20ffb
core/src/main/scala/kafka/api/OffsetFetchRequest.scala
c7604b9cdeb8f46507f04ed7d35fcb3d5f150713
core/src/main/scala/kafka/common/OffsetMetadataAndError.scala
1584a92447d276b6206d212b0b5487c352044154
core/src/main/scala/kafka/consumer/ZookeeperConsumerConnector.scala
cca815a128419e146feff53adaeddc901bb5de1f
core/src/main/scala/kafka/server/KafkaApis.scala
35af98f0bc1b6a50bd1d97a30147593f8c6a422d
core/src/main/scala/kafka/server/KafkaServer.scala
dddef938fabae157ed8644536eb1a2f329fb42b7
core/src/main/scala/kafka/server/OffsetManager.scala
d05e14d2018c0a0b5d22697313e9abde4363d65d
core/src/main/scala/kafka/server/ReplicaManager.scala
c5274822c57bf3c1f9e4135c0bdcaa87ee50ce20
core/src/test/scala/unit/kafka/api/RequestResponseSerializationTest.scala
fba852afa1b2f46b61e2fd12c38c821ba04e9cc6
core/src/test/scala/unit/kafka/server/OffsetCommitTest.scala
e4d0435eb4213597c2fb9c3f2093c227de53a417
core/src/test/scala/unit/kafka/server/ServerShutdownTest.scala
b46daa436231d5aa5c1e2992fd5c2d9a73a30c80
Diff: https://reviews.apache.org/r/27391/diff/
Testing
-------
Thanks,
Guozhang Wang