Skip to content

Update array operators for Group

Krasimir Angelov requested to merge 438124-update-array-operator-group into master

What does this MR do and why?

This is another change extracted from Convert integer ID columns to bigint (for new i... (!145752 - merged).

  • Use '{?}' as it casts to the type of the left side

  • Dynamically find the type of namespaces.traversal_ids to cast the right side

    This one is needed as there is no way to avoid explicit casting. There are places we currently use sql_type_metadata.sql_type (1, 2), but this is not safe as the metadata is cached, and once we actually swap the columns when migrating to bigint it may throw an error (/cc @minac).

This is needed in preparation for Use bigint for new installations (#438124 - closed), which is a prerequisite for Cells global identifiers.

See #438124 (comment 1833136499) for more context.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Related to #438124 (closed)

Merge request reports