ocsigen-start icon indicating copy to clipboard operation
ocsigen-start copied to clipboard

Split Os_db in Os_user_db, Os_group_db,...

Open dannywillems opened this issue 8 years ago • 2 comments

dannywillems avatar Dec 14 '16 21:12 dannywillems

Why? Users shouldn't really need to deal with Os_db, but with the higher-level abstractions (Os_user and similar). We don't get anything by intermittently moving code around. Also, "lots of micro-modules" has little to do with modularity.

vasilisp avatar Dec 14 '16 22:12 vasilisp

@vasilisp If more tables and more functions for the database are added in the future (I'm sure it would be the case), it will help to separate in different modules by purpose instead of having a big file for database.

Yes, users shouldn't need to deal with Os_db, but for us, I think it will be better to implement new table interface (it's done in bs).

dannywillems avatar Dec 14 '16 22:12 dannywillems