Remove unused @config declaration

The actually used declaration is in `StaticCatalogStoreConfig`.
ericluii · Apr 11, 2018 · 7b9c688 · 7b9c688
1 parent 4128cb5
commit 7b9c688
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 16 deletions.
diff --git a/presto-main/src/main/java/com/facebook/presto/server/PluginManagerConfig.java b/presto-main/src/main/java/com/facebook/presto/server/PluginManagerConfig.java
@@ -66,19 +66,6 @@ public PluginManagerConfig setPlugins(String plugins)
         return this;
     }
 
-    @NotNull
-    public File getPluginConfigurationDir()
-    {
-        return pluginConfigurationDir;
-    }
-
-    @Config("plugin.config-dir")
-    public PluginManagerConfig setPluginConfigurationDir(File pluginConfigurationDir)
-    {
-        this.pluginConfigurationDir = pluginConfigurationDir;
-        return this;
-    }
-
     @NotNull
     public String getMavenLocalRepository()
     {

diff --git a/presto-main/src/test/java/com/facebook/presto/server/TestPluginManagerConfig.java b/presto-main/src/test/java/com/facebook/presto/server/TestPluginManagerConfig.java
@@ -30,7 +30,6 @@ public void testDefaults()
         ConfigAssertions.assertRecordedDefaults(ConfigAssertions.recordDefaults(PluginManagerConfig.class)
                 .setInstalledPluginsDir(new File("plugin"))
                 .setPlugins((String) null)
-                .setPluginConfigurationDir(new File("etc/"))
                 .setMavenLocalRepository(ArtifactResolver.USER_LOCAL_REPO)
                 .setMavenRemoteRepository(ArtifactResolver.MAVEN_CENTRAL_URI));
     }
@@ -41,15 +40,13 @@ public void testExplicitPropertyMappings()
         Map<String, String> properties = new ImmutableMap.Builder<String, String>()
                 .put("plugin.dir", "plugins-dir")
                 .put("plugin.bundles", "a,b,c")
-                .put("plugin.config-dir", "plugin-configs")
                 .put("maven.repo.local", "local-repo")
                 .put("maven.repo.remote", "remote-a,remote-b")
                 .build();
 
         PluginManagerConfig expected = new PluginManagerConfig()
                 .setInstalledPluginsDir(new File("plugins-dir"))
                 .setPlugins(ImmutableList.of("a", "b", "c"))
-                .setPluginConfigurationDir(new File("plugin-configs"))
                 .setMavenLocalRepository("local-repo")
                 .setMavenRemoteRepository(ImmutableList.of("remote-a", "remote-b"));