trinodb / trino

Official repository of Trino, the distributed SQL query engine for big data, formerly known as PrestoSQL (https://trino.io)
https://trino.io
Apache License 2.0
10.28k stars 2.96k forks source link

Handle graceful shutdown on SIGTERM #1234

Open chancez opened 5 years ago

chancez commented 5 years ago

Presto should be able to handle signals and perform the appropriate shutdown tasks as needed.

From what I can tell, if you had separate initialization/run for https://github.com/prestosql/presto/blob/master/presto-main/src/main/java/io/prestosql/server/PrestoServer.java#L70 then Presto could:

oneonestar commented 5 years ago

Are there anyone looking into this issue? (cc: @electrum)

mm-matthias commented 5 months ago

This used to work for some time, but now it's broken with trino:446.