Skip to content

Follow up #3267#3285

Merged
ashie merged 2 commits into
fluent:masterfrom
ashie:followup-3267
Mar 8, 2021
Merged

Follow up #3267#3285
ashie merged 2 commits into
fluent:masterfrom
ashie:followup-3267

Conversation

@ashie

@ashie ashie commented Mar 8, 2021

Copy link
Copy Markdown
Member

Which issue(s) this PR fixes:
none

What this PR does / why we need it:
Some nitpicks for #3267

  • Suppress a warning while testing out_forward

    warning: instance variable @delayed_commit_timeout not initialized

  • Use more meaningful name for a method (set_interval -> ack_select_interval)

Docs Changes:
none

Release Note:
none

ashie added 2 commits March 8, 2021 09:22
set_interval ->
ack_select_interval

Signed-off-by: Takuro Ashie <ashie@clear-code.com>
  lib/fluent/plugin/out_forward.rb:298: warning: instance variable @delayed_commit_timeout not initialized

Signed-off-by: Takuro Ashie <ashie@clear-code.com>
@ashie ashie merged commit ba15b95 into fluent:master Mar 8, 2021
@ashie ashie deleted the followup-3267 branch March 8, 2021 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants