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

Remove Java 5 / JDBC 3.0 specific support [JDBC263] #310

Closed
firebird-automations opened this issue Aug 7, 2012 · 8 comments
Closed

Remove Java 5 / JDBC 3.0 specific support [JDBC263] #310

firebird-automations opened this issue Aug 7, 2012 · 8 comments

Comments

@firebird-automations
Copy link

Submitted by: @mrotteveel

Jaybird 2.3 will not support Java 5, so the Java 5 / JDBC 3.0 specific folders and ant properties can be removed. JDBC 4.0 classes and methods for features introduced in JDBC 4.0 can then be moved to the src/main-folder, and the content of src/wire_jdk1_4 and src/jca_jdk1_6 can also be moved into src/main

We also need to consider if it is a good or bad idea to move the full content of the src/jdbc4_0 folder into src/main and src/test4_0 to src/test and remove them.
Pros:
* more/everything in one place
* simpeler class and source hierarchy
cons:
* if Java 8 introduces new, incompatible changes we may need to reintroduce the folder
* potential loss of history (although this seems to be minor

Commits: 0809e0a 61c80b5 ef0d3c4 037a105 5061a31

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

Fix Version: Jaybird 2.3 [ 10440 ]

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

assignee: Roman Rokytskyy [ rrokytskyy ] => Mark Rotteveel [ avalanche1979 ]

@firebird-automations
Copy link
Author

Commented by: @mrotteveel

Moved contents of wire_jdk1_4 into main (this should have been done as part of JDBC195).

@firebird-automations
Copy link
Author

Commented by: @mrotteveel

Further changes pending final decision to remove Java 5 support

@firebird-automations
Copy link
Author

Commented by: @mrotteveel

Based on the survey results, we decided to go ahead and remove Java 5 support.

@firebird-automations
Copy link
Author

Commented by: @mrotteveel

Eliminated Java 5 support, and moved JDBC 4 classes into main and test. Also eliminated abstract classes no longer needed.

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

status: Resolved [ 5 ] => Closed [ 6 ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants