From b7a2c8d98e0b0ff4a2d731cfce078ab4587df48e Mon Sep 17 00:00:00 2001 From: Jaeho Yoo Date: Fri, 10 May 2024 11:31:00 +0900 Subject: [PATCH] Add additional log properties using config-properties file --- docs/installation.md | 11 ++++++----- gateway-ha/etc/config.properties | 4 ++++ .../java/io/trino/gateway/ha/HaGatewayLauncher.java | 4 ++-- 3 files changed, 12 insertions(+), 7 deletions(-) create mode 100644 gateway-ha/etc/config.properties diff --git a/docs/installation.md b/docs/installation.md index 3c729d2c5..935cc78fa 100644 --- a/docs/installation.md +++ b/docs/installation.md @@ -100,14 +100,15 @@ configuration YAML file. Find more information in the [routing rules documentation](routing-rules.md). - ### Configure logging -Path to `log.properties` must be set via `log.levels-file` JVM options -like `-Dlog.levels-file=etc/log.properties`. +To configure the logging level for various classes, specify the path to the +`log.properties` file using the `log.levels-file` JVM options, such as +`-Dlog.levels-file=etc/log.properties`, when the logging type is `external`. -Use the `log.*` properties from the [Trino logging properties -documentation](https://trino.io/docs/current/admin/properties-logging.html) for further configuration. +For additional configurations, use the `log.*` properties from the +[Trino logging properties documentation](https://trino.io/docs/current/admin/properties-logging.html) and specify the path to `config.properties` +file using the `config` JVM options, such as `-Dconfig=var/config.properties`. ### Proxying additional paths diff --git a/gateway-ha/etc/config.properties b/gateway-ha/etc/config.properties new file mode 100644 index 000000000..72bfb8b0f --- /dev/null +++ b/gateway-ha/etc/config.properties @@ -0,0 +1,4 @@ +% https://trino.io/docs/current/admin/properties-logging.html +log.max-size=100MB +log.max-total-size=1GB +log.compression=GZIP diff --git a/gateway-ha/src/main/java/io/trino/gateway/ha/HaGatewayLauncher.java b/gateway-ha/src/main/java/io/trino/gateway/ha/HaGatewayLauncher.java index 8ff771b0c..e1d492ee7 100644 --- a/gateway-ha/src/main/java/io/trino/gateway/ha/HaGatewayLauncher.java +++ b/gateway-ha/src/main/java/io/trino/gateway/ha/HaGatewayLauncher.java @@ -23,7 +23,7 @@ import java.util.Map; -import static io.airlift.configuration.ConfigurationLoader.getSystemProperties; +import static io.airlift.configuration.ConfigurationLoader.loadProperties; public class HaGatewayLauncher extends BaseApp @@ -40,7 +40,7 @@ public static void main(String[] args) throws Exception { Logging.initialize(); - Map properties = getSystemProperties(); + Map properties = loadProperties(); ConfigurationFactory configurationFactory = new ConfigurationFactory(properties); LoggingConfiguration configuration = configurationFactory.build(LoggingConfiguration.class); Logging logging = Logging.initialize();