-
Notifications
You must be signed in to change notification settings - Fork 4.1k
STORM-563. Kafka Spout doesn't pick up from the beginning of the queue unless forceFromStart specified. #493
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…e unless forceFromStart specified.
|
@ptgoetz @nathanmarz @revans2 Please take a look at the patch. I renamed forceFromStart to ignoreZkOffsets and users can configure where they want to start based on startOffsetTime . |
|
💯 |
external/storm-kafka/README.md
Outdated
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.
I would also document the actual values of EarliestTime() (-2) and LatestTime() (-1), and that it can also be set to a point in time (a la System.currentTimeMillis()).
My reasoning behind documenting the values (as opposed to the kafka API constants) is that the start offset time is likely to be specified via configuration (i.e. outside java code). Either that, or add spout constants that would get evaluated to EarliestTime()/LatestTime() if for some reason those values were ever changed in the Kafka API -- that seems like a less "leaky" solution.
…e unless forceFromStart specified.
|
@ptgoetz updated the doc as per your suggestion. Please take a look. |
|
@ptgoetz addressed your comments. Can you please take a look at this. |
|
It wouldn't hurt to expand on what |
|
That being said, I'm +1. |
|
If I set ignoreZkOffsets to true,will a spout recover from failure read from zk offsets or use startOffsetTime? |
|
@Renkai ignoreZkOffsets is rename of forcefromStart. So if you set ignoreZkOffsets it wil ignore already set offsets in zookeeper and start from the startOffsetTime. |
|
when ignoreZkOffsets set true and startOffsetTime = kafka.api.OffsetRequest.EarliestTime(). |
|
@Renkai In this case yes it won't read from zk offsets. Which is incorrect behavior. Can you file a jira on this. |
|
@harshach Issue created at https://issues.apache.org/jira/browse/STORM-1017 |
|
Thanks @Renkai |
No description provided.