Skip to content

chore(ci): cleanup for print statements, showing tables/schemas#17771

Open
the-other-tim-brown wants to merge 7 commits intoapache:masterfrom
the-other-tim-brown:test-code-cleanup
Open

chore(ci): cleanup for print statements, showing tables/schemas#17771
the-other-tim-brown wants to merge 7 commits intoapache:masterfrom
the-other-tim-brown:test-code-cleanup

Conversation

@the-other-tim-brown
Copy link
Contributor

@the-other-tim-brown the-other-tim-brown commented Jan 2, 2026

Describe the issue this Pull Request addresses

Cleans up tests to avoid excessive logging and removes unnecessary overrides in tests

Summary and Changelog

  • Remove overrides for test teardown when the test harness will already perform all the required cleanup
  • Remove printing in tests (println or show commands)
  • Cap size of error logs
  • Set jetty logs to warn level in CI

Impact

Make CI easier to debug and avoid noisy logs

Risk Level

None

Documentation Update

Contributor's checklist

  • Read through contributor's guide
  • Enough context is provided in the sections above
  • Adequate tests were added if applicable

@github-actions github-actions bot added the size:L PR with lines of changes in (300, 1000] label Jan 2, 2026
@the-other-tim-brown the-other-tim-brown changed the title chore(ci): cleanup for print statements, showing tables/schemas, direct GC calls chore(ci): cleanup for print statements, showing tables/schemas Jan 5, 2026
initHoodieStorage()
}

@AfterEach override def tearDown() = {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoiding overrides that match the default behavior to make it easier to make sweeping updates in the future

@the-other-tim-brown the-other-tim-brown marked this pull request as ready for review February 20, 2026 03:43
@hudi-bot
Copy link
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L PR with lines of changes in (300, 1000]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments