Skip to content

[IO- 889] Add test for Tailer close()#842

Merged
garydgregory merged 26 commits intoapache:masterfrom
garydgregory:bugfix/IO-889_tailer_close
Apr 8, 2026
Merged

[IO- 889] Add test for Tailer close()#842
garydgregory merged 26 commits intoapache:masterfrom
garydgregory:bugfix/IO-889_tailer_close

Conversation

@garydgregory
Copy link
Copy Markdown
Member

IO-889

Add test for Tailer's close().

Before you push a pull request, review this list:

  • Read the contribution guidelines for this project.
  • Read the ASF Generative Tooling Guidance if you use Artificial Intelligence (AI).
  • Run a successful build using the default Maven goal with mvn; that's mvn on the command line by itself.
  • Write unit tests that match behavioral changes, where the tests fail if the changes to the runtime are not applied. This may not always be possible, but it is a best practice.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Each commit in the pull request should have a meaningful subject line and body. Note that a maintainer may squash commits during the merge process.

@garydgregory garydgregory force-pushed the bugfix/IO-889_tailer_close branch from ff47119 to 04d7483 Compare April 8, 2026 11:31
- Add test class
- Green on macOS locally
@garydgregory garydgregory force-pushed the bugfix/IO-889_tailer_close branch from 04d7483 to 399d6c0 Compare April 8, 2026 11:33
- Add test class
- Green on macOS locally
- Fails on Windows on GH CI
@garydgregory garydgregory merged commit 2ab33b4 into apache:master Apr 8, 2026
21 checks passed
@garydgregory garydgregory deleted the bugfix/IO-889_tailer_close branch April 8, 2026 12:13
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.

1 participant