Skip to content

Introduce traversal ids8

Kamil Trzciński requested to merge introduce-traversal_ids8 into master

What does this MR do?

This addresses: #331230 (closed)

Changing:

  • adds traversal_ids8 in place of traversal_ids
  • changes all places to use traversal_ids8
  • fixes a lock update query to have a proper data type

Done:

  • I did check that backfill for root and children do work properly
  • I did check that namespace update works
  • I did check that linear traversal is properly used
  • All places and tests are covered

Not done:

  • It re-uses existing backfill to use traversal_ids8, maybe it requires a new one
  • I did not check query plans

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Kamil Trzciński

Merge request reports