#24 Committing Changes

[Sanjay Chaudhuri]

Hi Jason,

What is the best way to submit changes to the open issues. I am planning to go ahead with some of the fixes. Downloaded and built sources from SVN.

As a side note, I found that under com.amazonaws.eclipse.datatools.enablement.simpledb package, most of the Jdbc* java files are missing on mehtod implementations, and coming up with compilation errors.

Thanks

Sanjay

[Jason Fulghum]

Hey Sanjay,

it's great to hear that you're interested in learning more about contributing! Best thing to do would be chat with me and my team over e-mail to figure out a rough plan. Feel free to send me an e-mail (fulghum@amazon.com) and we can talk some more details on what you'd like to contribute.

Regarding the compilation errors on the Jdbc* java files...
You might double check that you're building against DTP-1.7. That's the first thing that comes to mind. I saw another developer run into some compilation problems there because he was developing in Eclipse-3.4 instead of Eclipse-3.5, and hadn't updated the DTP libraries in his Eclipse install.

[Sanjay Chaudhuri]

Hi Jason,

Yup; I am using DTP 1.7 as mentioned, and the whole application is working. Since I am not using Simple DB, so am not running into issues for the compilation errors after I built the application.

To be more specific here is an example
com.amazonaws.eclipse.datatools.enablement.simpledb.driver.JdbcConnection
implements java.sql.CallableStatement.Connection
and is missing the following 13 method implementations:
13 method(s) to implement:
- java.sql.Connection.createArrayOf()
- java.sql.Connection.createBlob()
- java.sql.Connection.createClob()
- java.sql.Connection.createNClob()
- java.sql.Connection.createSQLXML()
- java.sql.Connection.createStruct()
- java.sql.Connection.getClientInfo()
- java.sql.Connection.getClientInfo()
- java.sql.Connection.isValid()
- java.sql.Connection.setClientInfo()
- java.sql.Connection.setClientInfo()
- java.sql.Wrapper.isWrapperFor()
- java.sql.Wrapper.unwrap()

Thanks

Sanjay