Skip to content

Commit 5f2a1e2

Browse files
committed
Allow multiple directories as plugin source by extending the plugin.dir property to support multiple locations, comma separated
1 parent acc094e commit 5f2a1e2

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

core/trino-main/src/main/java/io/trino/server/ServerPluginsProviderConfig.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,15 +13,17 @@
1313
*/
1414
package io.trino.server;
1515

16+
import com.google.common.collect.ImmutableList;
1617
import io.airlift.configuration.Config;
1718
import io.airlift.configuration.ConfigDescription;
19+
import io.airlift.configuration.validation.FileExists;
1820

1921
import java.io.File;
2022
import java.util.List;
2123

2224
public class ServerPluginsProviderConfig
2325
{
24-
private List<File> installedPluginsDirs = List.of(new File("plugin"));
26+
private List<File> installedPluginsDirs = ImmutableList.of(new File("plugin"));
2527

2628
public List<File> getInstalledPluginsDirs()
2729
{
@@ -30,9 +32,9 @@ public List<File> getInstalledPluginsDirs()
3032

3133
@Config("plugin.dir")
3234
@ConfigDescription("Comma separated list of root directories where the plugins are located")
33-
public ServerPluginsProviderConfig setInstalledPluginsDirs(List<File> installedPluginsDirs)
35+
public ServerPluginsProviderConfig setInstalledPluginsDirs(List<@FileExists File> installedPluginsDirs)
3436
{
35-
this.installedPluginsDirs = installedPluginsDirs;
37+
this.installedPluginsDirs = ImmutableList.copyOf(installedPluginsDirs);
3638
return this;
3739
}
3840
}

0 commit comments

Comments
 (0)