DASSL / ClassDB

An open-source system to let students experiment with relational data
https://dassl.github.io/ClassDB/
Other
8 stars 2 forks source link

addConnectionMgmt.sql is incorrectly included in the core installation scripts (E) #142

Closed smurthys closed 6 years ago

smurthys commented 6 years ago

The script addConnectionMgmt.sql is included in prepareDB.psql though the functionality addConnectionMgmt.sql adds is not essential to using ClassDB: the functionality added is useful, but not essential.

I propose excluding addConnectionMgmt.sql from prepardDB.psql.

smurthys commented 6 years ago

I also propose the following improvements :

Proposed organization for prepareDBAddOn.psql:

smurthys commented 6 years ago

Also, we should classify each script file as "Core" or "AddOn" and add the file to appropriate .psql file

smurthys commented 6 years ago

We made the following the decision in the Skype call on Jan. 10, 2017 about this topic:

In addition to the structure summarized above, I propose server-level scripts and db-level scripts be stored in separate sub-directories, and each of them further containing kind-specific sub-directories. A kind-specific sub-directory may be not created at all if no script of that kind is present.

The following is the proposed directory structure for source files:

src
  -- server
    -- core
    -- reco
    -- opt
  -- db
    -- core
    -- reco
    -- opt
afig commented 6 years ago

Resolved by #164