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

Deployment trino-server-445 NullPointerException #21669

Open
jxlining opened this issue Apr 23, 2024 · 9 comments
Open

Deployment trino-server-445 NullPointerException #21669

jxlining opened this issue Apr 23, 2024 · 9 comments

Comments

@jxlining
Copy link

jxlining commented Apr 23, 2024

/opt/programs/trino-server/etc/resource-groups.properties
resource-groups.configuration-manager=db
resource-groups.config-db-url=jdbc:mysql://xxxxx:3306/resource_groups_445
resource-groups.config-db-user=mytrino
resource-groups.config-db-password=xxxxx

2024-04-23T17:47:03.808+0800 INFO main io.trino.plugin.resourcegroups.db.FlywayMigration Performed 0 migrations
2024-04-23T17:47:03.819+0800 ERROR main io.trino.server.Server null
java.lang.NullPointerException
at java.base/java.util.Objects.requireNonNull(Objects.java:233)
at com.google.common.collect.Maps.fromProperties(Maps.java:1429)
at io.airlift.configuration.ConfigurationLoader.getSystemProperties(ConfigurationLoader.java:70)
at io.airlift.bootstrap.Bootstrap.configure(Bootstrap.java:177)
at io.airlift.bootstrap.Bootstrap.initialize(Bootstrap.java:246)
at io.trino.plugin.resourcegroups.db.DbResourceGroupConfigurationManagerFactory.create(DbResourceGroupConfigurationManagerFactory.java:57)
at io.trino.execution.resourcegroups.InternalResourceGroupManager.setConfigurationManager(InternalResourceGroupManager.java:162)
at io.trino.execution.resourcegroups.InternalResourceGroupManager.loadConfigurationManager(InternalResourceGroupManager.java:146)
at io.trino.server.Server.doStart(Server.java:174)
at io.trino.server.Server.lambda$start$0(Server.java:94)
at io.trino.$gen.Trino_445____20240423_094644_1.run(Unknown Source)
at io.trino.server.Server.start(Server.java:94)
at io.trino.server.TrinoServer.main(TrinoServer.java:38)

not using the resource groups properties method is success

Tasks

Preview Give feedback
No tasks being tracked yet.
@jxlining
Copy link
Author

trino-403 using the resource groups properties issuccess

@akenO8
Copy link
Member

akenO8 commented Apr 30, 2024

I thought it was the polyglot.engine.WarnInterpreterOnly 's value was set incorrectly, not string, cause NullPointerException.
<dep.flyway.version>10.11.0</dep.flyway.version>

Add -Dpolyglot.engine.WarnInterpreterOnly=false to jvm.config.

@ebyhr
That seems to be the reason. flyway/flyway#3866 and flyway/flyway#3882

@jxlining It looks like it will be fixed in trino-446 (flyway-10.11.1), so let's look forward to 446~

@ebyhr
Copy link
Member

ebyhr commented May 20, 2024

Closing as #21641. Please reopen if this issue still happens with newer versions.

@ebyhr ebyhr closed this as completed May 20, 2024
@findepi
Copy link
Member

findepi commented May 20, 2024

I believe the problem remains.

@findepi findepi reopened this May 20, 2024
@findepi
Copy link
Member

findepi commented May 20, 2024

This should help: airlift/airlift#1167

@findepi
Copy link
Member

findepi commented May 22, 2024

airlift change merged. let's close this issue when we upgrade airlift.

@akenO8
Copy link
Member

akenO8 commented May 28, 2024

I think your pr does not solve this problem.
When the initial system properties have null, it is inevitable. Should we check or avoid these illegal system properties in getSystemProperties()?

@findepi
Copy link
Member

findepi commented May 28, 2024

@akenO8 System.setProperty("xxx", null) fails on machine. How can i enter the state when initial system properties have null value?

@akenO8
Copy link
Member

akenO8 commented May 29, 2024

image
I'm sorry I didn't understand you.
As far as validate properties are concerned, I think we can add a method to remove validate property. I implemented it and avoided the exception of this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants