[
https://issues.apache.org/jira/browse/STORM-352?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14031025#comment-14031025
]
ASF GitHub Bot commented on STORM-352:
--------------------------------------
Github user d2r commented on the pull request:
https://github.com/apache/incubator-storm/pull/142#issuecomment-46049088
+1
Makes sense to me based on diff here:
https://github.com/apache/incubator-storm/commit/5b0258322fd0b479fb372a69184a24bc27958c21#diff-f031b5b4822879d6ba8aaf42f74a4aa7L195
> [storm-kafka] PartitionManager does not save offsets to ZooKeeper
> -----------------------------------------------------------------
>
> Key: STORM-352
> URL: https://issues.apache.org/jira/browse/STORM-352
> Project: Apache Storm (Incubating)
> Issue Type: Bug
> Affects Versions: 0.9.2-incubating
> Reporter: P. Taylor Goetz
> Assignee: P. Taylor Goetz
> Labels: kafka
>
> The logic to determine whether offsets should be written to ZK is wrong. As a
> result, offsets are never saved.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
