-
Notifications
You must be signed in to change notification settings - Fork 4.1k
distsql: support tuples in streams #15938
Copy link
Copy link
Closed
Labels
A-sql-executionRelating to SQL execution.Relating to SQL execution.C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)
Milestone
Description
We currently don't support tuples in DistSQL streams. We don't have a ColumnType_Kind for it and no way to encode it over the wire. This makes certain queries non-distributed.
For example: SELECT (k, v) FROM t.
Another example is more subtle: SELECT k, v FROM t ORDER BY (k, v). Here the extra parens cause a tuple to be created for each row and used as the sort key. So this query is not supported by DistSQL, as opposed to the equivalent SELECT k, v FROM t ORDER BY k, v. Granted, this case should be probably converted to latter version before distsql planning.
CC @asubiotto
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-sql-executionRelating to SQL execution.Relating to SQL execution.C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)