Changes between Initial Version and Version 4 of Ticket #8647
- Timestamp:
- Sep 11, 2012, 10:31:18 AM (12 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #8647
- Property Owner set to
- Property Milestone → 1.0.1
- Property Keywords documentation added
-
Ticket #8647 – Description
initial v4 1 This patch simplifies Trac DB API to make it easier to extend and add new DB backends. It introduces abstract DatabaseConnectionclass that clearly defines what required methods are used by Trac and must be implemented by Connection classes.1 This patch simplifies Trac DB API to make it easier to extend and add new DB backends. It introduces abstract `DatabaseConnection` class that clearly defines what required methods are used by Trac and must be implemented by Connection classes. 2 2 3 I've made it, because I found implicit ConnectionWrapper behavior misleading. I used SQLiteConnection method set as an example, but in the end my implementation failed to execute, because it didn't have commit() method - just like SQLiteConnection. It took a while to dig through PooledConnection to ConnectionWrapper then to SQLiteConnection and finally to ConnectionWrapperagain to realize that commit() method is implicitly called from bundled SQLite library when it isn't found anywhere else.3 I've made it, because I found implicit `ConnectionWrapper` behavior misleading. I used `SQLiteConnection` method set as an example, but in the end my implementation failed to execute, because it didn't have commit() method - just like `SQLiteConnection`. It took a while to dig through `PooledConnection` to `ConnectionWrapper` then to `SQLiteConnection` and finally to `ConnectionWrapper` again to realize that commit() method is implicitly called from bundled SQLite library when it isn't found anywhere else. 4 4 5 5 I hope this negative developer experience will find more response in tracker than it had in mailing list. =)