GitHub user HeartSaVioR opened a pull request:
https://github.com/apache/storm/pull/1251
STORM-1654 HBaseBolt creates tick tuples with no interval when we don't set
flushIntervalSecs
Please also refer #893 to see why this change is necessary.
Set 'default' flush interval seconds to HBaseBolt, since taking half of
message timeout secs doesn't work, as #893 showed.
Since we should hardcode default flush interval seconds for now, I think we
can change default flush interval to 1s since I guess it's enough for HBase,
and anyone never set up tuple timeout seconds to 1. (When if this is happening,
all bolts with batching manner using tick tuple should also fail.)
I'll craft separate PR to be applied to 1.x-branch.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/HeartSaVioR/storm STORM-1654
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/1251.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 #1251
----
commit c6ed8d0c419dba8bdb479a432d5e7855f78b012b
Author: Jungtaek Lim <[email protected]>
Date: 2016-03-24T02:57:28Z
STORM-1654 HBaseBolt creates tick tuples with no interval when we don't set
flushIntervalSecs
* set 'default' flush interval seconds (1s) to HBaseBolt
* since taking half of message timeout secs doesn't work
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
