diff --git a/presto-local-file/src/main/java/io/prestosql/plugin/localfile/LocalFileConfig.java b/presto-local-file/src/main/java/io/prestosql/plugin/localfile/LocalFileConfig.java index b2a161f2f..e3a93fd6d 100644 --- a/presto-local-file/src/main/java/io/prestosql/plugin/localfile/LocalFileConfig.java +++ b/presto-local-file/src/main/java/io/prestosql/plugin/localfile/LocalFileConfig.java @@ -32,6 +32,8 @@ public class LocalFileConfig return maxLogItemsLimit; } + @Config("presto-logs.http-request-log.location") + @ConfigDescription("Directory or file where http request logs are written") public LocalFileConfig setHttpRequestLogLocation(String httpRequestLogLocation) { this.httpRequestLogLocation = httpRequestLogLocation; diff --git a/presto-local-file/src/test/java/io/prestosql/plugin/localfile/TestLocalFileConfig.java b/presto-local-file/src/test/java/io/prestosql/plugin/localfile/TestLocalFileConfig.java index 54f79c3d5..688ed97ae 100644 --- a/presto-local-file/src/test/java/io/prestosql/plugin/localfile/TestLocalFileConfig.java +++ b/presto-local-file/src/test/java/io/prestosql/plugin/localfile/TestLocalFileConfig.java @@ -25,7 +25,7 @@ public class TestLocalFileConfig public void testDefaults() { ConfigAssertions.assertRecordedDefaults(ConfigAssertions.recordDefaults(LocalFileConfig.class) - .setHttpRequestLogFileNamePattern(null)); + .setHttpRequestLogFileNamePattern(null).setHttpRequestLogLocation("var/log/http-request.log")); } @Test @@ -33,11 +33,14 @@ public class TestLocalFileConfig { Map properties = new ImmutableMap.Builder() .put("presto-logs.http-request-log.pattern", "bar") + .put("presto-logs.http-request-log.location", "jar") .build(); LocalFileConfig expected = new LocalFileConfig() .setHttpRequestLogFileNamePattern("bar"); + expected.setHttpRequestLogLocation("jar"); + ConfigAssertions.assertFullMapping(properties, expected); } }