NorthArrowResearch / champ-workbench

http://workbench.northarrowresearch.com/
GNU General Public License v3.0
0 stars 0 forks source link

Metric Copy Tool Produces Empty Batches #56

Closed philipbaileynar closed 6 years ago

philipbaileynar commented 6 years ago

The tool that copies metrics from multiple batches into a new batch is not performing the actual step that copies the actual metric values. This tool performs three steps:

  1. Create the new batch
  2. Copy all instances from each of the existing batches into the new batch
  3. Copy all the metric values from each of the existing batches into the new batch

The final step is written and present in the code but the SQL statements are written as SELECT statements instead of `INSERT INTO. I suspect that this was written this way during development and debugging so that I could review the data that was going to get copied without actually making any permanent data changes to the underlying database. I then forgot to finish each SQL statement by converting them into insert statements.