Skip to content

Conversation

@marschall
Copy link

Deprecate Derby support since Apache Derby is retired.

Closes gh-48567

I did no log any warnings when Derby support is used. I'm not aware of such a thing being done anywhere today, let me know if I should add it.

I marked only main code for removal, no tests. I had to add @SuppressWarnings to main and test code using deprecated Derby code for the build to work.

I believe this should go to the 4.1 changelog. I did find a changelog checked in. Are these manually maintained?

Deprecate Derby support since Apache Derby is retired.

Closes spring-projectsgh-48567

Signed-off-by: Philippe Marschall <[email protected]>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Dec 19, 2025
Copy link
Member

@snicoll snicoll left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, I've made a few comments for your consideration

* Apache Derby.
* @deprecated Derby is retired, use HSQLDB or H2
*/
@Deprecated(forRemoval = true, since = "4.1")
Copy link
Member

Choose a reason for hiding this comment

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

This should be 4.1.0


/**
* Apache Derby.
* @deprecated Derby is retired, use HSQLDB or H2
Copy link
Member

Choose a reason for hiding this comment

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

Please look at the other deprecation in the codebase, we use a standard format for this.

}

@Test
@SuppressWarnings("removal")
Copy link
Member

Choose a reason for hiding this comment

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

You should flag the test as deprecated instead;

@snicoll snicoll added the status: waiting-for-feedback We need additional information before we can continue label Dec 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: waiting-for-feedback We need additional information before we can continue status: waiting-for-triage An issue we've not yet triaged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecate Derby support since Apache Derby is retired

3 participants