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.

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

pg17beta2 support #1792

Merged
merged 9 commits into from
Aug 12, 2024
Merged

Conversation

eeeebbbbrrrr
Copy link
Contributor

Starting on pg17beta2 support...

Some tests fail locally. I dunno if these passed before -- I didn't run the test suite first. I assume they didn't. Will investigate.

Not sure what'll happen on CI, which is why I'm PRing this now:

test tests::proptests::tests::pg_date_literal_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_timetz_literal_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_timestamp_literal_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_time_literal_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_date_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_timetz_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_time_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_timestamp_spi_roundtrip ... FAILED

@eeeebbbbrrrr
Copy link
Contributor Author

eeeebbbbrrrr commented Jul 27, 2024

interesting. Those tests also fail for me locally from develop using pg16. What's up with that? Any idea @workingjubilee?

They all have the same failure pattern:

Postgres Messages:
[2024-07-27 11:58:23.173 EDT] [1608822] [66a5191f.188c76]: LOG:  starting PostgreSQL 16.3 on x86_64-pc-linux-gnu, compiled by gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0, 64-bit
[2024-07-27 11:58:23.173 EDT] [1608822] [66a5191f.188c76]: LOG:  listening on IPv4 address "127.0.0.1", port 32216
[2024-07-27 11:58:23.175 EDT] [1608822] [66a5191f.188c76]: LOG:  listening on Unix socket "/home/zombodb/_work/pgrx/target/test-pgdata/.s.PGSQL.32216"
[2024-07-27 11:58:23.181 EDT] [1608822] [66a5191f.188c76]: LOG:  database system is ready to accept connections


Test Function Messages:
[2024-07-27 11:58:23.467 EDT] [1611050] [66a5191f.18952a]: LOG:  statement: BEGIN
[2024-07-27 11:58:23.467 EDT] [1611050] [66a5191f.18952a]: LOG:  statement: SELECT "tests"."date_spi_roundtrip"();
[2024-07-27 11:58:23.467 EDT] [1611050] [66a5191f.18952a]: ERROR:  function tests.date_spi_roundtrip() does not exist at character 8
[2024-07-27 11:58:23.467 EDT] [1611050] [66a5191f.18952a]: HINT:  No function matches the given name and argument types. You might need to add explicit type casts.
[2024-07-27 11:58:23.467 EDT] [1611050] [66a5191f.18952a]: STATEMENT:  SELECT "tests"."date_spi_roundtrip"();
[2024-07-27 11:58:23.467 EDT] [1611050] [66a5191f.18952a]: LOG:  statement: ROLLBACK


Client Error:
function tests.date_spi_roundtrip() does not exist
postgres location: parse_func.c
rust location: <unknown>

I'm running cargo test --all --features pg16,cshim --no-default-features from the root of the repo. Am I missing something?

@eeeebbbbrrrr
Copy link
Contributor Author

Am I missing something?

YES! I was missing the proptest feature. Okay. Something to look at later...

@eeeebbbbrrrr
Copy link
Contributor Author

tests pass locally now on both linux and MacOS. A+++

I'm doing the CI dance now to get it to install/use the 17 beta deb packages.

@eeeebbbbrrrr eeeebbbbrrrr self-assigned this Jul 27, 2024
@eeeebbbbrrrr eeeebbbbrrrr marked this pull request as ready for review July 27, 2024 17:49
@eeeebbbbrrrr eeeebbbbrrrr changed the title WIP: working on pg17beta2 support. working on pg17beta2 support. Jul 30, 2024
@eeeebbbbrrrr
Copy link
Contributor Author

As far as I can tell, this is ready to merge

@workingjubilee
Copy link
Member

@eeeebbbbrrrr This needs a rebase.

Some tests fail locally.  Not sure what'll happen on CI, which is why I'm PRing this now:

```
test tests::proptests::tests::pg_date_literal_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_timetz_literal_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_timestamp_literal_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_time_literal_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_date_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_timetz_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_time_spi_roundtrip ... FAILED
test tests::proptests::tests::pg_timestamp_spi_roundtrip ... FAILED
```
@eeeebbbbrrrr
Copy link
Contributor Author

done

@workingjubilee
Copy link
Member

oh nice.

@workingjubilee workingjubilee changed the title working on pg17beta2 support. pg17beta2 support Aug 12, 2024
Copy link
Member

@workingjubilee workingjubilee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@workingjubilee workingjubilee merged commit 6dfb9d1 into pgcentralfoundation:develop Aug 12, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants