Issue Details (XML | Word | Printable)

Key: JRUBY-2438
Type: New Feature New Feature
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Nick Sieger
Reporter: Joe Athman
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
JRuby

Support SQLite3 using JRuby

Created: 24/Apr/08 12:30 AM   Updated: 04/Jul/08 09:07 AM
Component/s: ActiveRecord-JDBC
Affects Version/s: None
Fix Version/s: JRuby 1.1.2

Time Tracking:
Not Specified

File Attachments: 1. Text File sqlite3_adapter.2.patch (3 kB)
2. Text File sqlite3_adapter.patch (19 kB)

Environment: Mac OSX 10.5.2

Testcase included: yes
Patch Submitted: Yes


 Description  « Hide
I think there should be support for the SQLite3 database using JRuby and ActiveRecord-JDBC since SQLite is now the default DB of Rails. I'm attaching a patch which is at least a good starting point for the driver.

All simple tests pass now, except for the rename column test since this feature is not natively supported in SQLite3.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Nick Sieger added a comment - 24/Apr/08 10:32 AM
You might start looking to Rails native sqlite3 adapter for inspiration on how to solve some of the remaining issues.

Nick Sieger added a comment - 24/Apr/08 10:44 AM
Thanks for working on this, this is really cool!

I committed the first patch; please pull from subversion and merge any changes you have if you could, please.

Leaving the ticket open for now, just keep attaching patches if you have more work.


Joe Athman added a comment - 24/Apr/08 10:07 PM
here's another patch to apply. This implements the remove_column functionality and cleans up some other stuff. Now it will run all of the simple tests.

Thomas E Enebo added a comment - 12/May/08 04:01 PM
Assigning to Nick. Also indicating that it would be nice for this to be in AR-JDBC around same time we release 1.1.2 (it would be great to have working sqlite3 support for this)

Nick Sieger added a comment - 13/May/08 09:22 PM
Thanks again for the patch. If you have any more, just open a new issue.