Bug 197806 - Maxdb connectivity contribution
Summary: Maxdb connectivity contribution
Status: CLOSED FIXED
Alias: None
Product: Data Tools
Classification: Tools
Component: Enablement:SAP (show other bugs)
Version: 1.5   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: 1.6M5   Edit
Assignee: Wolfgang Auer CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-07-25 10:31 EDT by Dimiter Dimitrov CLA
Modified: 2008-07-01 17:39 EDT (History)
4 users (show)

See Also:
jgraham: pmc_approved+
sholars: pmc_approved+
lchan: pmc_approved+
bfitzpat: review+


Attachments
MAXDB database definition and specific enhancements (10.78 KB, application/x-zip-compressed)
2007-10-04 15:49 EDT, Dimiter Dimitrov CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dimiter Dimitrov CLA 2007-07-25 10:31:53 EDT
With this request I want to offer contribution - connectivity support for MAXDB database. I have prepared two plugins – one for the database definition (XMI) and one for specific MAXDB implementation – DDL generator, builder and small enhancements, specific for MAXDB. The plugins are tested in SAP developer studio. If you are interested about such contribution, please tell me and I will attach the patches with the relevant plugins.
Comment 1 Brian Fitzpatrick CLA 2007-07-26 14:16:43 EDT
We would be pleased to review your contribution as an attachment to the bugzilla entry. 
--Brian Fitzpatrick (DTP Connectivity)
Comment 2 Dimiter Dimitrov CLA 2007-10-04 15:49:28 EDT
Created attachment 79769 [details]
MAXDB database definition and specific enhancements
Comment 3 Dimiter Dimitrov CLA 2007-10-04 15:53:13 EDT
Hello Brian,

At last we received permission to provide MAXDB XMI to DTP. The attached ZIP (see comment#2)contains all we want to contribute – XMI, which describes MAXDB and enhancements, specific for MAXDB. The structure of the attached folder is plugin structure. In the applied DBConfiguration and TableConstraintLoader I have described (as comment) what the purpose of the implementation is, and which bugzilla entry we have workarounded. If you have some questions, don’t hesitate to contact me. The separation of XMI definition and implementation if it’s necessary I’m leaving on you.
From my side I have only one question. We haven’t implemented tests for this stuff yet and unfortunately I couldn’t find exemplary tests in CVS for other database enablement plugins. If you have requirements for test coverage, please give me and example and/or short description which kind of testing you prefer : unit test, functionality test, etc.

Best Regards,
Dimiter
Comment 4 John Graham CLA 2007-10-04 16:03:13 EDT
Dimiter,

Thank you for this contribution! We'll start the Eclipse review procedure for bringing this code into the DTP code line, and certainly will get back to you with status and questions.
Comment 5 Dimiter Dimitrov CLA 2007-10-05 04:20:21 EDT
Hello John,

I forgot to mention, that we prefer to contribute this stuff for Ganymede release. The reason is, that it's immature yet and we plan to develop it during the following months.

Best Regards,
Dimiter

Comment 6 Brian Fitzpatrick CLA 2007-10-17 11:01:33 EDT
Hey Dimiter...

Your MaxDB contribution looks great. A couple of things to note... 

1) We're still planning on supporting JDK 1.4.x, not just JDK 1.5, so all JDK 1.5 features (like <> and @override indicators) need to be removed
2) The assert class you use in MaxDbTableConstraintLoader should be org.eclipse.core.runtime.Assert. I think this is also a JDK 1.5 thing. 

Other than that, it worked and looked fine. We appreciate the help in the enablement area adding additional DBMS types!

--Fitz
Comment 7 Brian Fitzpatrick CLA 2007-10-17 11:01:50 EDT
Approved
Comment 8 John Graham CLA 2007-10-25 11:32:40 EDT
Pending Eclipse IP review and determination of committer resources.
Comment 9 John Graham CLA 2008-01-09 16:51:56 EST
IP review completed by Eclipse legal. Wolfgang Auer nominated as committer for this component.
Comment 10 John Graham CLA 2008-01-10 12:48:31 EST
Asking for Brian's review w.r.t the Enablement "best practices" doc

Initial tm for 1.6 set, pending committer election, build integration and testing.
Comment 11 Brian Fitzpatrick CLA 2008-01-16 17:41:12 EST
Hi John...

Based on what's in this contribution, it appears pretty straightforward to get it in line with the enablement doc... The one plug-in would be split into two plug-ins.

The dbdefinition and its extension would go into its own plug-in (o.e.d.e.sap.maxdb.dbdefinition) and you could simply rename the o.e.d.e.maxdb plugin to match the form that Larry established for best practices: o.e.d.e.sap.maxdb.

It doesn't appear that a o.e.d.e.sap.maxdb.ui plug-in is necessary for this stage in development, but could be created with a readme.txt indicating that that is where any custom UI-related code would go.

Let me know if you have any other questions, but this should be pretty straightforward to do.

--Fitz
Comment 12 John Graham CLA 2008-01-24 15:11:36 EST
Assigning to Wolfgang, since he is now the component lead for this.
Comment 13 Wolfgang Auer CLA 2008-01-30 07:29:16 EST
A inital contribution for MaxDB is done.
Comment 14 Brian Fitzpatrick CLA 2008-07-01 17:39:46 EDT
Closing bugs