-
Notifications
You must be signed in to change notification settings - Fork 4.1k
opt: change the key column of inverted indexes to use a new column id and type #50695
Copy link
Copy link
Closed
Closed
Copy link
Labels
A-spatialSpatial work that is *not* related to builtins.Spatial work that is *not* related to builtins.A-sql-optimizerSQL logical planning and optimizations.SQL logical planning and optimizations.C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.
Description
Currently, the inverted key column of an inverted index uses the same column ID and type as the original table column it indexes. This is a problem if the column needs to be passed between several operators, because the execution engine expects a column with a certain type (e.g., Geometry), but receives a different type (e.g., int). It also makes several things awkward in the optimizer, and requires special handling of the inverted key column.
The solution is to create a new column for the inverted key column, with a new column ID and type corresponding to the real type stored in the inverted index. This issue tracks the work to make that change in the optimizer.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-spatialSpatial work that is *not* related to builtins.Spatial work that is *not* related to builtins.A-sql-optimizerSQL logical planning and optimizations.SQL logical planning and optimizations.C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.