[ 
https://issues.apache.org/jira/browse/STORM-643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14632249#comment-14632249
 ] 

ASF GitHub Bot commented on STORM-643:
--------------------------------------

GitHub user vesense opened a pull request:

    https://github.com/apache/storm/pull/642

    [STORM-643] KafkaUtils repeatedly fetches messages whose offset is out of 
range

    STORM-643 fixing based on master branch

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/vesense/storm patch-4

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/642.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #642
    
----
commit 1a0a3c2bfe4ace4c3febb14fccbda8af4e61cfa5
Author: Xin Wang <[email protected]>
Date:   2015-07-18T03:39:51Z

    STORM-643 fixing based on master branch

commit 9400850f849d59354d4c6a2d7786f1e23e268cf2
Author: Xin Wang <[email protected]>
Date:   2015-07-18T03:46:08Z

    STORM-643 fixing based on master branch

----


> KafkaUtils repeatedly fetches messages whose offset is out of range (for 
> 0.9.x)
> -------------------------------------------------------------------------------
>
>                 Key: STORM-643
>                 URL: https://issues.apache.org/jira/browse/STORM-643
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: storm-kafka
>    Affects Versions: 0.9.2-incubating, 0.9.3, 0.9.4, 0.9.5
>            Reporter: Xin Wang
>            Assignee: Xin Wang
>             Fix For: 0.9.6
>
>
> KafkaUtils repeat fetch messages which offset is out of range.
> This happened when failed list(SortedSet<Long> failed) is not empty and some 
> offset in it is OutOfRange.
> [worker-log]
> {code}
> 2015-02-01 10:24:27.231+0800 s.k.KafkaUtils [WARN] Got fetch request with 
> offset out of range: [20919071816]; retrying with default start offset time 
> from configuration. configured start offset time: [-2]
> 2015-02-01 10:24:27.232+0800 s.k.PartitionManager [WARN] Using new offset: 
> 20996130717
> 2015-02-01 10:24:27.333+0800 s.k.KafkaUtils [WARN] Got fetch request with 
> offset out of range: [20919071816]; retrying with default start offset time 
> from configuration. configured start offset time: [-2]
> 2015-02-01 10:24:27.334+0800 s.k.PartitionManager [WARN] Using new offset: 
> 20996130717
> ...
> {code}
> [FIX]
> {code}
> storm.kafka.PartitionManager.fill():
> ...
> try {
>       msgs = KafkaUtils.fetchMessages(_spoutConfig, _consumer, _partition, 
> offset);
> } catch (UpdateOffsetException e) {
>        _emittedToOffset = KafkaUtils.getOffset(_consumer, _spoutConfig.topic, 
> _partition.partition, _spoutConfig);
>       LOG.warn("Using new offset: {}", _emittedToOffset);
>       // fetch failed, so don't update the metrics
>       //fix bug: remove this offset from failed list when it is OutOfRange
>       if (had_failed) {
>               failed.remove(offset);
>       }
>             return;
> }
> ...
> {code}
> also: Log "retrying with default start offset time from configuration. 
> configured start offset time: [-2]" is incorrect.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to