cockroachdb / cockroach

CockroachDB - the open source, cloud-native distributed SQL database.
https://www.cockroachlabs.com
Other
29.49k stars 3.7k forks source link

roachtest: pg_regress failed #124381

Open cockroach-teamcity opened 1 month ago

cockroach-teamcity commented 1 month ago

roachtest.pg_regress failed with artifacts on master @ 93ad913106b6f0f6ec98bc2cfa788ff6d8085bd4:

    RETURN NULL;
  END;
  $$;
@@ -239345,7 +240368,7 @@
  --
  -- Test multiple reference to a transition table
  --
-@@ -5761,22 +6524,45 @@
+@@ -5761,22 +6550,45 @@
         FROM (SELECT * FROM new_test UNION ALL SELECT * FROM new_test) ss);
      RETURN NULL;
  END$$;
@@ -239393,7 +240416,7 @@
  CREATE OR REPLACE FUNCTION get_from_partitioned_table(partitioned_table.a%type)
  RETURNS partitioned_table AS $$
  DECLARE
-@@ -5787,13 +6573,13 @@
+@@ -5787,13 +6599,13 @@
      SELECT * INTO result FROM partitioned_table WHERE a = a_val;
      RETURN result;
  END; $$ LANGUAGE plpgsql;
@@ -239413,7 +240436,7 @@
  CREATE OR REPLACE FUNCTION list_partitioned_table()
  RETURNS SETOF partitioned_table.a%TYPE AS $$
  DECLARE
-@@ -5806,14 +6592,13 @@
+@@ -5806,14 +6618,13 @@
      END LOOP;
      RETURN;
  END; $$ LANGUAGE plpgsql;
@@ -239434,7 +240457,7 @@
  --
  -- Check argument name is used instead of $n in error message
  --
-@@ -5822,6 +6607,15 @@
+@@ -5822,6 +6633,15 @@
    GET DIAGNOSTICS x = ROW_COUNT;
    RETURN;
  END; $$ LANGUAGE plpgsql;
@@ -265846,7 +266869,7 @@
 +       ^
 +HINT:  try \h CREATE
  create table rewriteme (id serial primary key, foo float, bar timestamptz);
-+NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/v24.1/serial.html
++NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/v24.2/serial.html
  insert into rewriteme
       select x * 1.001 from generate_series(1, 500) as t(x);
  alter table rewriteme alter column foo type numeric;
If the diff is expected, copy regression.diffs from the test artifacts to pkg/cmd/roachtest/testdata/regression.diffs
test artifacts and logs in: /artifacts/pg_regress/run_1

Parameters:

See: roachtest README

See: How To Investigate (internal)

See: Grafana

This test on roachdash | Improve this report!

Jira issue: CRDB-38856

cockroach-teamcity commented 1 month ago

roachtest.pg_regress failed with artifacts on master @ b5081e997adaf5950c9b85dfb6655f226bf16d29:

    RETURN NULL;
  END;
  $$;
@@ -239345,7 +240378,7 @@
  --
  -- Test multiple reference to a transition table
  --
-@@ -5761,22 +6524,45 @@
+@@ -5761,22 +6550,45 @@
         FROM (SELECT * FROM new_test UNION ALL SELECT * FROM new_test) ss);
      RETURN NULL;
  END$$;
@@ -239393,7 +240426,7 @@
  CREATE OR REPLACE FUNCTION get_from_partitioned_table(partitioned_table.a%type)
  RETURNS partitioned_table AS $$
  DECLARE
-@@ -5787,13 +6573,13 @@
+@@ -5787,13 +6599,13 @@
      SELECT * INTO result FROM partitioned_table WHERE a = a_val;
      RETURN result;
  END; $$ LANGUAGE plpgsql;
@@ -239413,7 +240446,7 @@
  CREATE OR REPLACE FUNCTION list_partitioned_table()
  RETURNS SETOF partitioned_table.a%TYPE AS $$
  DECLARE
-@@ -5806,14 +6592,13 @@
+@@ -5806,14 +6618,13 @@
      END LOOP;
      RETURN;
  END; $$ LANGUAGE plpgsql;
@@ -239434,7 +240467,7 @@
  --
  -- Check argument name is used instead of $n in error message
  --
-@@ -5822,6 +6607,15 @@
+@@ -5822,6 +6633,15 @@
    GET DIAGNOSTICS x = ROW_COUNT;
    RETURN;
  END; $$ LANGUAGE plpgsql;
@@ -265846,7 +266879,7 @@
 +       ^
 +HINT:  try \h CREATE
  create table rewriteme (id serial primary key, foo float, bar timestamptz);
-+NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/v24.1/serial.html
++NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/v24.2/serial.html
  insert into rewriteme
       select x * 1.001 from generate_series(1, 500) as t(x);
  alter table rewriteme alter column foo type numeric;
If the diff is expected, copy regression.diffs from the test artifacts to pkg/cmd/roachtest/testdata/regression.diffs
test artifacts and logs in: /artifacts/pg_regress/run_1

Parameters:

See: roachtest README

See: How To Investigate (internal)

See: Grafana

This test on roachdash | Improve this report!

mgartner commented 3 weeks ago

I think we just need to change the NOTICE line from 24.1 to 24.2.

-+NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/v24.1/serial.html
++NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/v24.2/serial.html
cockroach-teamcity commented 3 weeks ago

roachtest.pg_regress failed with artifacts on master @ c7f37e813dbd1e6d2afdb88c5e9cd986dc061299:

+@@ -5761,22 +6553,45 @@
         FROM (SELECT * FROM new_test UNION ALL SELECT * FROM new_test) ss);
      RETURN NULL;
  END$$;
@@ -239393,7 +240431,7 @@
  CREATE OR REPLACE FUNCTION get_from_partitioned_table(partitioned_table.a%type)
  RETURNS partitioned_table AS $$
  DECLARE
-@@ -5787,13 +6573,13 @@
+@@ -5787,13 +6602,13 @@
      SELECT * INTO result FROM partitioned_table WHERE a = a_val;
      RETURN result;
  END; $$ LANGUAGE plpgsql;
@@ -239413,7 +240451,7 @@
  CREATE OR REPLACE FUNCTION list_partitioned_table()
  RETURNS SETOF partitioned_table.a%TYPE AS $$
  DECLARE
-@@ -5806,14 +6592,13 @@
+@@ -5806,14 +6621,13 @@
      END LOOP;
      RETURN;
  END; $$ LANGUAGE plpgsql;
@@ -239434,7 +240472,7 @@
  --
  -- Check argument name is used instead of $n in error message
  --
-@@ -5822,6 +6607,15 @@
+@@ -5822,6 +6636,16 @@
    GET DIAGNOSTICS x = ROW_COUNT;
    RETURN;
  END; $$ LANGUAGE plpgsql;
@@ -239442,6 +240480,7 @@
 -LINE 3:   GET DIAGNOSTICS x = ROW_COUNT;
 -                          ^
 +ERROR:  unimplemented: attempted to use a PL/pgSQL statement that is not yet supported
++DETAIL:  stmt_get_diag is not yet supported
 +HINT:  You have attempted to use a feature that is not yet implemented.
 +
 +Please check the public issue tracker to check whether this problem is
@@ -265846,7 +266885,7 @@
 +       ^
 +HINT:  try \h CREATE
  create table rewriteme (id serial primary key, foo float, bar timestamptz);
-+NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/v24.1/serial.html
++NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/v24.2/serial.html
  insert into rewriteme
       select x * 1.001 from generate_series(1, 500) as t(x);
  alter table rewriteme alter column foo type numeric;
If the diff is expected, copy regression.diffs from the test artifacts to pkg/cmd/roachtest/testdata/regression.diffs
test artifacts and logs in: /artifacts/pg_regress/run_1

Parameters:

See: roachtest README

See: How To Investigate (internal)

See: Grafana

This test on roachdash | Improve this report!

cockroach-teamcity commented 2 weeks ago

roachtest.pg_regress failed with artifacts on master @ 3aa5bdf40527e2f60b179094403b4302a1c2cbe1:

++NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/dev/serial.html
  insert into rewriteme
       select x * 1.001 from generate_series(1, 500) as t(x);
  alter table rewriteme alter column foo type numeric;
 -ERROR:  rewrites not allowed
 -CONTEXT:  PL/pgSQL function test_evtrig_no_rewrite() line 3 at RAISE
 +ERROR:  ALTER COLUMN TYPE from float to decimal is only supported experimentally
-+HINT:  See: https://go.crdb.dev/issue-v/49329/v24.2
++HINT:  See: https://go.crdb.dev/issue-v/49329/dev
 +--
 +you can enable alter column type general support by running ``SET enable_experimental_alter_column_type_general = true``
  alter table rewriteme add column baz int default 0;
@@ -265870,7 +266901,7 @@
   alter column foo type numeric(10,4);
 -NOTICE:  Table 'rewriteme' is being rewritten (reason = 4)
 +ERROR:  ALTER COLUMN TYPE from float to decimal is only supported experimentally
-+HINT:  See: https://go.crdb.dev/issue-v/49329/v24.2
++HINT:  See: https://go.crdb.dev/issue-v/49329/dev
 +--
 +you can enable alter column type general support by running ``SET enable_experimental_alter_column_type_general = true``
  -- matview rewrite when changing access method
@@ -265892,7 +266923,7 @@
  -- shouldn't trigger a table_rewrite event
  alter table rewriteme alter column foo type numeric(12,4);
 +ERROR:  ALTER COLUMN TYPE from float to decimal is only supported experimentally
-+HINT:  See: https://go.crdb.dev/issue-v/49329/v24.2
++HINT:  See: https://go.crdb.dev/issue-v/49329/dev
 +--
 +you can enable alter column type general support by running ``SET enable_experimental_alter_column_type_general = true``
  begin;
@@ -265932,7 +266963,7 @@
 +alter type rewritetype alter attribute a type text cascade
 +                                                          ^
 +HINT:  You have attempted to use a feature that is not yet implemented.
-+See: https://go.crdb.dev/issue-v/48701/v24.2
++See: https://go.crdb.dev/issue-v/48701/dev
  -- but this doesn't work
  create table rewritemetoo3 (a rewritetype);
  alter type rewritetype alter attribute a type varchar cascade;
@@ -265942,7 +266973,7 @@
 +alter type rewritetype alter attribute a type varchar cascade
 +                                                             ^
 +HINT:  You have attempted to use a feature that is not yet implemented.
-+See: https://go.crdb.dev/issue-v/48701/v24.2
++See: https://go.crdb.dev/issue-v/48701/dev
  drop table rewriteme;
  drop event trigger no_rewrite_allowed;
 +ERROR:  at or near "event": syntax error
If the diff is expected, copy regression.diffs from the test artifacts to pkg/cmd/roachtest/testdata/regression.diffs
test artifacts and logs in: /artifacts/pg_regress/run_1

Parameters:

See: roachtest README

See: How To Investigate (internal)

See: Grafana

This test on roachdash | Improve this report!

cockroach-teamcity commented 1 week ago

roachtest.pg_regress failed with artifacts on master @ 8a97a5edd98336e2dd04ef12f08628fba84b17dd:

++NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/dev/serial.html
  insert into rewriteme
       select x * 1.001 from generate_series(1, 500) as t(x);
  alter table rewriteme alter column foo type numeric;
 -ERROR:  rewrites not allowed
 -CONTEXT:  PL/pgSQL function test_evtrig_no_rewrite() line 3 at RAISE
 +ERROR:  ALTER COLUMN TYPE from float to decimal is only supported experimentally
-+HINT:  See: https://go.crdb.dev/issue-v/49329/v24.2
++HINT:  See: https://go.crdb.dev/issue-v/49329/dev
 +--
 +you can enable alter column type general support by running ``SET enable_experimental_alter_column_type_general = true``
  alter table rewriteme add column baz int default 0;
@@ -265870,7 +266881,7 @@
   alter column foo type numeric(10,4);
 -NOTICE:  Table 'rewriteme' is being rewritten (reason = 4)
 +ERROR:  ALTER COLUMN TYPE from float to decimal is only supported experimentally
-+HINT:  See: https://go.crdb.dev/issue-v/49329/v24.2
++HINT:  See: https://go.crdb.dev/issue-v/49329/dev
 +--
 +you can enable alter column type general support by running ``SET enable_experimental_alter_column_type_general = true``
  -- matview rewrite when changing access method
@@ -265892,7 +266903,7 @@
  -- shouldn't trigger a table_rewrite event
  alter table rewriteme alter column foo type numeric(12,4);
 +ERROR:  ALTER COLUMN TYPE from float to decimal is only supported experimentally
-+HINT:  See: https://go.crdb.dev/issue-v/49329/v24.2
++HINT:  See: https://go.crdb.dev/issue-v/49329/dev
 +--
 +you can enable alter column type general support by running ``SET enable_experimental_alter_column_type_general = true``
  begin;
@@ -265932,7 +266943,7 @@
 +alter type rewritetype alter attribute a type text cascade
 +                                                          ^
 +HINT:  You have attempted to use a feature that is not yet implemented.
-+See: https://go.crdb.dev/issue-v/48701/v24.2
++See: https://go.crdb.dev/issue-v/48701/dev
  -- but this doesn't work
  create table rewritemetoo3 (a rewritetype);
  alter type rewritetype alter attribute a type varchar cascade;
@@ -265942,7 +266953,7 @@
 +alter type rewritetype alter attribute a type varchar cascade
 +                                                             ^
 +HINT:  You have attempted to use a feature that is not yet implemented.
-+See: https://go.crdb.dev/issue-v/48701/v24.2
++See: https://go.crdb.dev/issue-v/48701/dev
  drop table rewriteme;
  drop event trigger no_rewrite_allowed;
 +ERROR:  at or near "event": syntax error
If the diff is expected, copy regression.diffs from the test artifacts to pkg/cmd/roachtest/testdata/regression.diffs
test artifacts and logs in: /artifacts/pg_regress/run_1

Parameters:

See: roachtest README

See: How To Investigate (internal)

See: Grafana

This test on roachdash | Improve this report!

cockroach-teamcity commented 2 days ago

roachtest.pg_regress failed with artifacts on master @ 047a7ed79756eef53b8b9ab4c9dd9c5a463496c9:

        select 1, 1, 1
       ) s(a, b, c)
  where s.a = 1 and s.b = 1 and s.c = (select 1);
@@ -261212,7 +261200,7 @@
  select *
  from (
        select * from p
-@@ -3769,11 +3286,7 @@
+@@ -3769,11 +3285,7 @@
        select 1, 1, 1
       ) s(a, b, c)
  where s.a = 1 and s.b = 1 and s.c = (select 1);
@@ -261225,7 +261213,7 @@
  prepare q (int, int) as
  select *
  from (
-@@ -3784,116 +3297,186 @@
+@@ -3784,116 +3296,186 @@
        select 1, 1, 1
       ) s(a, b, c)
  where s.a = $1 and s.b = $2 and s.c = (select 1);
@@ -261480,7 +261468,7 @@
  -- Like the above but throw some more complexity at the planner by adding
  -- a UNION ALL.  We expect both sides of the union not to scan the
  -- non-required partitions.
-@@ -3901,136 +3484,236 @@
+@@ -3901,136 +3483,236 @@
  'select * from listp where a = (select 1)
    union all
  select * from listp where a = (select 2);');
@@ -261793,7 +261781,7 @@
  --
  -- Check that gen_partprune_steps() detects self-contradiction from clauses
  -- regardless of the order of the clauses (Here we use a custom operator to
-@@ -4043,28 +3726,84 @@
+@@ -4043,28 +3725,84 @@
     commutator = ===,
     hashes
  );
@@ -265846,7 +265834,7 @@
 +       ^
 +HINT:  try \h CREATE
  create table rewriteme (id serial primary key, foo float, bar timestamptz);
-+NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/v24.1/serial.html
++NOTICE:  using sequential values in a primary key does not perform as well as using random UUIDs. See https://www.cockroachlabs.com/docs/v24.2/serial.html
  insert into rewriteme
       select x * 1.001 from generate_series(1, 500) as t(x);
  alter table rewriteme alter column foo type numeric;
If the diff is expected, copy regression.diffs from the test artifacts to pkg/cmd/roachtest/testdata/regression.diffs
test artifacts and logs in: /artifacts/pg_regress/run_1

Parameters:

See: roachtest README

See: How To Investigate (internal)

See: Grafana

This test on roachdash | Improve this report!