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

Add a Jetty event listener to start Derby when Roller is run via mvn jetty:run #131

Merged
merged 2 commits into from
Sep 30, 2023

Conversation

snoopdave
Copy link
Contributor

This PR fixes a regression caused by switching Roller from using the Maven btmatthews inmemdb plugin and instead starting Derby by calling the embedded Derby server API directly (which works when tests are run either in IntelliJ or in Maven). The problem was that I forgot to add the same logic to the Maven Jetty plugin so that Derby is started when Roller is run via mvn jetty:run.

The Jira ticket: https://issues.apache.org/jira/browse/ROL-2182

@snoopdave snoopdave merged commit fd17702 into master Sep 30, 2023
5 of 6 checks passed
@mbien
Copy link
Member

mbien commented Sep 30, 2023

unfortunately this breaks the integration tests :(

@snoopdave
Copy link
Contributor Author

Bummer. I'll take another look over the weekend. I thought I had fixed the IT tests, except for the jdk21 run.

@mbien
Copy link
Member

mbien commented Sep 30, 2023

@snoopdave oh you are right - I looked at the wrong workflow run! All good, my mistake.

the jdk 21 run is easy to fix. its just the setup action requesting an early access build - i can fix this later.

edit: opened #132

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