Pre-CR review - Section B comments

Section B1

* The classname of "BaseTableOrView" stands out as particularly cumbersome.
I would change this to "SQLTable". I would let the "or view" part be
defined in the formal definition but understood in the type name. I would
also change R2RMLView to SQLQuery. This would lead to a tight, clear set of
type names: a LogicalTable could be either a SQLTable or SQLQuery.

Received on Monday, 23 January 2012 19:51:30 UTC