Skip to content

Add test class progress logging on CI#11651

Merged
hashhar merged 1 commit intotrinodb:masterfrom
findinpath:test-class-progress-logging
Mar 25, 2022
Merged

Add test class progress logging on CI#11651
hashhar merged 1 commit intotrinodb:masterfrom
findinpath:test-class-progress-logging

Conversation

@findinpath
Copy link
Copy Markdown
Contributor

@findinpath findinpath commented Mar 24, 2022

While going through the CI logs it can be helpful to pinpoint
the specific test class issuing errors on the logs.

Description

While going through the CI logs it can be helpful to pinpoint
the specific test class issuing errors on the logs.
If the JVM suddenly crashes while running the @BeforeClass setup methods, with the help of the current changes
it can be seen which specific class is causing these issue and reduce therefore the time needed to reproduce locally
the issue from the CI server.

Is this change a fix, improvement, new feature, refactoring, or other?

Improvement

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

Testing infrastructure.

How would you describe this change to a non-technical end user or system administrator?

Pinpoint the test class causing JVM crashes while setting up the testing infrastructure for its test methods.

Look for:

  • BEFORE CLASS
  • AFTER CLASS

in the CI logs corresponding to the build of this PR.

Related issues, pull requests, and links

Documentation

(x) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required.
( ) Release notes entries required with the following suggested text:

While going through the CI logs it can be helpful to pinpoint
the specific test class issuing errors on the logs.
@cla-bot cla-bot bot added the cla-signed label Mar 24, 2022
@findinpath findinpath requested review from hashhar and losipiuk March 24, 2022 09:49
@hashhar
Copy link
Copy Markdown
Member

hashhar commented Mar 25, 2022

Hit #11634 - unrelated.

@hashhar hashhar added the no-release-notes This pull request does not require release notes entry label Mar 25, 2022
@hashhar hashhar merged commit 422a176 into trinodb:master Mar 25, 2022
@github-actions github-actions bot added this to the 375 milestone Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed no-release-notes This pull request does not require release notes entry

Development

Successfully merging this pull request may close these issues.

3 participants