IVOA Standard SQL/ADQL reserved terms used for CAOM Table names in CAOM 2.5

[ebrassfieldbourke]

As I understand it, part of CAOM version 2.5 is the intention to recommend CAOM as an IVOA standard. CAOM 2.5 currently uses SQL and ADQL reserved words for table names such as Position and Time. A workaround for reserved words is to place these table names into quotation marks. Perhaps the issue/fix could be added to the CAOM documentation upon IVOA Standardisation?
If this is a known issue or I missed it folded into the other issues, my apologies, and please feel free to delete or mark as 'Resolved'.

Output from a TAP Lint tool run by my colleague:
Section TME: Check content of tables metadata from /tables E-TME-TRSV-1 Table name is ADQL reserved word 'Position' E-TME-TRSV-2 Table name is ADQL reserved word 'Time' E-TME-CRSV-1 Column name is ADQL reserved word 'pi' in table Proposal - should delimit like '"pi"' E-TME-CRSV-2 Column name is ADQL reserved word 'coordsys' in table SpatialWCS - should delimit like '"coordsys"' E-TME-CRSV-3 Column name is ADQL reserved word 'coordsys' in table TargetPosition - should delimit like '"coordsys"'

Error also results from using a simple 'SELECT' command on these tables.

[pdowler]

I take it that this is based on a database and TAP service where all tables were generated by @pahjbo vo-dml tools? There will definitely be discussion around model -> db schema related to #28

Still, use of names that are reserved words in some context is risky at least and tweaks to prevent future pain are a good idea. See #29 for an example.

---

Short blurb: those are not intended to end up being table names, but since the model comes before the standard relational mapping spec, it is possible someone would implement db tables like the boxes in the UML diagram. That's not necessary since a 0..1 relation can just be flattened into a set of columns in the "parent" table.

The intent is that entity classes map to tables (so currently 6 tables in total, 8 when PR #27 is done). That may drop back to 5 tables if we decide to completely remove part and chunk.