SUMMARY OF CODE REVIEW -- 08/21/1996: Top-level Functionality of the Execution Engine - exec.h/c, ops.c

Presenter: Ramana Yerneni
Note-taker: Joachim Hammer

Items of discussion:

In exec.h/c
  1. Inclusion of btable.h not necessary since the graph traversal functions have been moved. The same is true for class bTable.
  2. Rename headtemp and tailtemp to avoid confusion with wrapper templates. Suggested replacement names are qhead and qtail.
  3. sourceanswer can only be deleted along with the answer.
  4. Investigate the difference bwteen NULL bindings and empty table.
  5. In case of an error, delete bindings before the function break.

In ops.c

  1. Need function for cleaning up result tables (for every node in the graph)
  2. colindex is superfluous; just cehck for -1

Joachim Hammer