Skip to content

Inconsistency: $sqlquery-export accepts ViewDefinition table sources but $sqlquery-run does not #383

Description

@johngrimes

The $sqlquery-export operation has a repeating view parameter (with name, viewReference and viewResource parts) for supplying the ViewDefinitions that a SQLQuery Library's relatedArtifact entries depend on. $sqlquery-run has no equivalent, so there is no way to satisfy a query's ViewDefinition dependencies inline when running synchronously.

Looking at the history, the view parameter appears to have been introduced when $sqlquery-export was added (48920a8, closing #319) rather than as a deliberate design decision - #319 only asked for an async mirror of $sqlquery-run plus the export-control parameters from $viewdefinition-export, and $sqlquery-run was never reconciled afterwards.

If anything, the asymmetry is backwards: the synchronous run operation is the natural home for the ad-hoc workflow (inline queryResource plus inline ViewDefinitions, nothing stored on the server), yet it is the one operation that cannot accept views.

Whichever shape the view parameter ends up taking (#384 questions the value of viewReference and the undefined binding semantics), the two operations should agree: either both accept ViewDefinition table sources or neither does.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions