Status: In Progress
Fix Version/s: None
Sprint:DB_F20_09, DB_S21_12, DB_F21_06, DB_S22_12, DB_F22_6, DB_S23_6
Team:Data Access and Database
Please clarify whether the operational DAX+Qserv system should be queried using the qserv_* family of functions (UDFs) or the corresponding scisql_* functions.
Example: qserv_areaspec_circle versus scisql_s2PtInCircle.
From documentation I can see, the scisql_* functions may be a (substantial) superset of the qserv_* functions.
I understand that the answer to this question may be different for the 2016-17 PDAC versus the planned final system.
I believe so (yes, now irrelevant).
TAP notwithstanding, Qserv itself also now automatically infers the most common area restrictions directly from `scisql_s2PtIn...`.
Gregory Dubois-Felsmann: please set status to "Invalid" if you feel the issue is satisfactorily resolved, or clarify required further action here if not?
Kian-Tat Lim: Was the status update to "In Review" on 09/Nov/16 intentional? I do not see any linked ticket. Is/was there material requiring review?
As I said at the time, the status was updated to "In Review" because I was asserting that existing documentation already clarified this policy and wanted a check on that.
Thanks, Kian-Tat Lim (sorry I missed your original comment above the fold). I'll attach this to our current F20 doc epic, and close it when we've updated UserManual.md wrt. auto-inferred spatial constraints.
It seems like this ticket is still relevant in that we are allowing scisql calls directly into the ADQL.
We need to make a decision about this, maybe on a new ticket, since we have to be intentional about extensions to ADQL given:
- We are promising to support those extensions for a long time since people want their queries to work.
- We need to minimize these extensions (do not use scisql for sky area when ADQL works fine)
- We need to whitelist specific extensions.
- We should consider renaming them so they are not locking us into scisql if at some point we change from mariadb.
Has this ticket become irrelevant now that we are using ADQL?