Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log the H2 DB server status to provide more details on its state and URL #4171

Merged
merged 1 commit into from Sep 24, 2019
Merged

Log the H2 DB server status to provide more details on its state and URL #4171

merged 1 commit into from Sep 24, 2019

Conversation

jaikiran
Copy link
Member

A very minor change to print out the "status" of the H2 DB when it's started and stopped, so that details like the following gets printed when running integration tests:

[INFO] H2 database started in TCP server mode; server status: TCP server running at tcp://localhost:9092 (only local connections)

This additional info would have helped me when I ran into an issue trying to run the integration tests locally (more details on quarkus-dev mailing list)

@gsmet gsmet added this to the 0.24.0 milestone Sep 24, 2019
Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Looks like a useful addition. Thanks!

Let's wait for CI.

@gsmet gsmet added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 24, 2019
@mkouba mkouba merged commit 0bfbabd into quarkusio:master Sep 24, 2019
@jaikiran jaikiran deleted the h2-test-log branch July 4, 2020 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/waiting-for-ci Ready to merge when CI successfully finishes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants