Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Derive the session.id/order_number from the upload.id #720

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Derive the session.id/order_number from the upload.id #720

wants to merge 3 commits into from

Commits on Sep 18, 2024

  1. Make session.id == upload.id (#713)

    @Swatinem
    Swatinem committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    f91cede View commit details
    Browse the repository at this point in the history

  2. Try clamping an upload order_number to a smaller int 

    The problem here is that `order_number` is a 32-bit signed int (effectively 31-bit),
and duplicating the `reports_upload.id` into it does not work, as that value already exceeds 31-bits in production.

This tries to work around that problem by arranging that number to occupy bits 12-28.
    @Swatinem
    Swatinem committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    67be052 View commit details
    Browse the repository at this point in the history

  3. fix various foreign key column definitions

    @Swatinem
    Swatinem committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    cdf4bba View commit details
    Browse the repository at this point in the history