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

[Bug report] Failed to create/get schema in postgres 9.2 (use postgres as database) #1786

Closed
danhuawang opened this issue Jan 30, 2024 · 4 comments · Fixed by #1805
Closed
Assignees
Labels
bug Something isn't working

Comments

@danhuawang
Copy link
Contributor

Version

main branch

Describe what's wrong

截屏2024-01-30 17 29 40 截屏2024-01-30 17 24 14 截屏2024-01-30 17 38 48

Error message and/or stacktrace

{
"code": 1002,
"type": "RuntimeException",
"message": "Failed to operate object [pg_temp_1] operation [LOAD] under [pg2], reason [ERROR: type "regnamespace" does not exist\n 位置:37]",
"stack": [
"java.lang.RuntimeException: com.datastrato.gravitino.exceptions.GravitinoRuntimeException: ERROR: type "regnamespace" does not exist",
" 位置:37",
"\tat com.datastrato.gravitino.catalog.CatalogOperationDispatcher.doWithCatalog(CatalogOperationDispatcher.java:752)",
"\tat com.datastrato.gravitino.catalog.CatalogOperationDispatcher.loadSchema(CatalogOperationDispatcher.java:208)",
"\tat com.datastrato.gravitino.server.web.rest.SchemaOperations.lambda$loadSchema$2(SchemaOperations.java:115)",
"\tat java.base/java.security.AccessController.doPrivileged(AccessController.java:712)",
"\tat java.base/javax.security.auth.Subject.doAs(Subject.java:439)",
"\tat com.datastrato.gravitino.utils.PrincipalUtils.doAs(PrincipalUtils.java:25)",
"\tat com.datastrato.gravitino.server.web.Utils.doAs(Utils.java:121)",
"\tat com.datastrato.gravitino.server.web.rest.SchemaOperations.loadSchema(SchemaOperations.java:111)",
"\tat jdk.internal.reflect.GeneratedMethodAccessor19.invoke(Unknown Source)",
"\tat java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)",
"\tat java.base/java.lang.reflect.Method.invoke(Method.java:568)",
"\tat org.glassfish.jersey.server.model.internal.ResourceMethodInvocationHandlerFactory.lambda$static$0

How to reproduce

  1. launch postgres 9.2
  2. create catalog in gravitino
    jdbc-url: jdbc:postgresql://localhost:5433/postgres
    jdbc-user: pgtest
    jdbc-driver: org.postgresql.Driver
    jdbc-password: pgtest
    jdbc-database: postgres
  3. get the exist schema info or create a new schema by api

Additional context

No response

@danhuawang danhuawang added the bug Something isn't working label Jan 30, 2024
@FANNG1
Copy link
Contributor

FANNG1 commented Jan 30, 2024

I suggest to remove the support for pg 9, for this version is too old and not supported for now. please refer https://help.aliyun.com/zh/rds/apsaradb-rds-for-postgresql/lifecycles-of-major-engine-versions.
And in our integrate test only tests pg from 12 to 16 which is supported for now.

@jerryshao jerryshao added this to the Gravitino 0.4.0 milestone Jan 30, 2024
@danhuawang
Copy link
Contributor Author

@justinmclean
Copy link
Member

Please refer to the official documentation https://www.postgresql.org/support/versioning/ final version dates:
9.2 November 9, 2017
12.0 November 14, 2024
13.0 November 13, 2025

It's unclear from above, but are versions 14,15 and 16 supported? If so, then the documentation should be updated.

@FANNG1
Copy link
Contributor

FANNG1 commented Jan 31, 2024

https://help.aliyun.com/zh/rds/apsaradb-rds-for-postgresql/lifecycles-of-major-engine-versions

Our doc writes Supports metadata management of PostgreSQL (9.2, 12.0, 13.0) https://datastrato.ai/docs/0.3.1/jdbc-postgresql-catalog

the document should be corrected, I will post a PR

jerryshao pushed a commit that referenced this issue Jan 31, 2024
### What changes were proposed in this pull request?
1. update PG supported version
2. add warning messages not change gravitino id
3. add not support set MySQL scheme comment message

### Why are the changes needed?


Fix: #1786 #1704 

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
no, just document
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants