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.

Sign up for GitHub

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

Jump to bottom

Mark methods removed in Jaybird 6 as deprecated in Jaybird 5 #759

Closed
mrotteveel opened this issue Jul 21, 2023 · 1 comment
Closed

Mark methods removed in Jaybird 6 as deprecated in Jaybird 5 #759

mrotteveel opened this issue Jul 21, 2023 · 1 comment
Assignees
@mrotteveel
Labels
type: task
Milestone
v5.0.3

Comments

@mrotteveel
Copy link
Member

Jaybird 6 removes quite a number of methods and classes (most of them internal), lets mark them as deprecated in Jaybird 5 so people who do rely on them are aware.

NOTE: This ticket is specifically for 5.0.3, if any other methods get removed after 5.0.3 release, a new ticket will be created.
@mrotteveel mrotteveel added this to the v5.0.3 milestone Jul 21, 2023
@mrotteveel mrotteveel self-assigned this Jul 21, 2023
mrotteveel added a commit that referenced this issue Jul 21, 2023
@mrotteveel
#759 Mark classes/methods/constants removed in Jaybird 6 as deprecated
9137856
mrotteveel added a commit that referenced this issue Jul 21, 2023
@mrotteveel
#759 Older Java versions don't allow @deprecated in package-info.java
b1f8f88
mrotteveel added a commit that referenced this issue Jul 25, 2023
@mrotteveel
#759 We retroactively marked some methods as deprecated in Jaybird 5.0.3
579b428
mrotteveel added a commit that referenced this issue Jul 26, 2023
@mrotteveel
#759 Mark more classes/methods/constants removed in Jaybird 6 as depr…
00de4b9 
…ecated
@mrotteveel
Copy link
Member Author

I decided not to mark the deprecation of FirebirdStatement#getCurrentResultSet from #762 in Jaybird 5 (5.0.3), because the suggested replacement of using Statement#getResultSet will throw an exception in Jaybird 5 if the result set was already returned.

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

@mrotteveel mrotteveel

Labels
type: task
Projects
None yet
Milestone
v5.0.3
Development

No branches or pull requests
1 participant
@mrotteveel