| Summary: | MySQL OSGi Support | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [RT] Gemini.DBaccess | Reporter: | Tom Ware <tom.ware> | ||||||
| Component: | Core | Assignee: | Project Inbox <gemini.dbaccess-inbox> | ||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||
| Severity: | enhancement | ||||||||
| Priority: | P3 | CC: | gunnar, juergen.kissner, michael.keith | ||||||
| Version: | unspecified | Flags: | juergen.kissner:
iplog+
|
||||||
| Target Milestone: | --- | ||||||||
| Hardware: | PC | ||||||||
| OS: | Windows XP | ||||||||
| Whiteboard: | |||||||||
| Attachments: |
|
||||||||
|
Description
Tom Ware
Correction: We will likely need a CQ for the MySQL JDBC driver. Created attachment 181288 [details]
Initial Changes - without MySQL jdbc driver
Created attachment 182752 [details]
Updated fix - no PDE project for MySQL driver
I had a look at the attachment: the current bundle and DBAccess class structure requires quite a bit of code replication with blows up the necessary code. As a side effect the number of lines exceeds 250. If I understand the Eclipse Legal process correctly, I have to file a CQ, because we do not work for the same company (even if both companies have signed the Member Committer agreement). Am I wrong on that? So I wonder, if we should try and refactor the org.eclipse.gemini.dbaccess bundles before integrating the MySQL contribution. There are also additional questions about: - the possible central build - the tests which, in the current state, would run on Derby only. What do you think? There is definitely alot of shared code and the refactor will help alot. You are correct that for you to check this in there would have to be a CQ. I spoke to Mike Keith about this and because he and I work for the same company (and that company has signed the Eclipse agreement), no CQ would be required if he checked it in - I was under the impression that he was planning on doing that if it was ok with you. I am ok with whatever sequence of steps works for you. Yes, I will check the code into DBAccess SVN if that works for you, Juergen. Then we can do the refactoring at your discretion. Sorry, I was playing with mylyn and muddeled up the Bugs MySQL has been included through revisions http://dev.eclipse.org/viewcvs/viewvc.cgi?view=revision&root=RT_GEMINI.DBACCESS&revision=79 http://dev.eclipse.org/viewcvs/viewvc.cgi?view=revision&root=RT_GEMINI.DBACCESS&revision=80 http://dev.eclipse.org/viewcvs/viewvc.cgi?view=revision&root=RT_GEMINI.DBACCESS&revision=81 |