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

Some installation files are missing or have incorrect names (WM) #185

Closed wildtayne closed 6 years ago

wildtayne commented 6 years ago

There are no installation scripts for installing all components at once, and one installation script has an incorrect name. A proposed solution is:

  1. [x] prepareServerCore.sql => initializeServerCore.sql
  2. [x] Add addAllDB.psql, which installs all DB components
  3. [x] Add addAllServer.psql which installs all server components

The two new installer scripts should call the existing addAllXY.psql scripts instead of individual sql files.

smurthys commented 6 years ago

Related to this issue, as discussed yesterday, the Quick Start doc also needs to be changed to show installation of all scripts.

wildtayne commented 6 years ago

One other thing I've noticed:

smurthys commented 6 years ago

Renaming removeAllFrom[DB/Server] to removeAll[DB/Server] seems OK.

smurthys commented 6 years ago

In retrospect, I worry removeAll[DB/Server] could be misconstrued as meaning "remove all DBs/servers". Maybe the better approach is to say addAlTo[DB/Server], even though it has a different form than the other add scripts.

I am sorry I did not state this possibility earlier (too much multi-tasking go on).