Uploaded image for project: 'FOLIO'
  1. FOLIO
  2. FOLIO-2978

Remove uuid_generate_v4(), it fails in pgpool native replication

    XMLWordPrintable

Details

    • Bug
    • Status: Open (View Workflow)
    • P4
    • Resolution: Unresolved
    • None
    • None
    • CP: Non-roadmap backlog
    • Core: Platform

    Description

      uuid_generate_v4() creates a different UUID in each node of a replicated PostgreSQL environment if Pgpool-II runs in native replication mode.
      This has been reported for the similar function gen_random_uuid() with Pgpool-II replication in #sys-ops on Slack. From the Pgpool-II manual:
      https://www.pgpool.net/docs/latest/en/html/restrictions.html

      There is no guarantee that any data provided using a context-dependent mechanism (e.g. random number, transaction ID, OID, SERIAL, sequence), will be replicated correctly on multiple backends.

      RMB documentation mentions it in the upgrading guide for RMB 25 and in connection with "generateId": https://github.com/folio-org/raml-module-builder/#the-post-tenant-api

      Tasks:

      Note:
      For predefined records use a hard-coded UUID.
      For calculated records re-use the UUID of the primary source record.
      Or use the md5sum trick: If a string (for example a JSONB::text) is unique calculate md5sum(s)::text::uuid to get a unique UUID.

      To prevent any new usage of uuid_generate_v4() RMB and Spring base should add this function that takes precedence of the uuid-ossp function:

      CREATE OR REPLACE FUNCTION public.uuid_generate_v4() RETURNS uuid
      language plpgsql as $$
      BEGIN
        RAISE 'FOLIO doesn''t allow uuid_generate_v4() because it doesn''t work with a replicated database like Pgpool-II'
          USING ERRCODE = 'feature_not_supported';
      END;
      $$;

      Note:

      This doesn't affect Pgpool-II streaming replication mode, it only affect Pgpool-II native replication mode.

      TestRail: Results

        Attachments

          Issue Links

            Activity

              People

                Unassigned Unassigned
                julianladisch Julian Ladisch
                Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                  Created:
                  Updated:

                  TestRail: Runs

                    TestRail: Cases