Move FunctionNamespaceManager to SPI

prestodb · Sep 4, 2019 · 85d30a6 · 85d30a6
1 parent 5149d92
commit 85d30a6
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 9 deletions.
diff --git a/presto-main/src/main/java/com/facebook/presto/metadata/HandleResolver.java b/presto-main/src/main/java/com/facebook/presto/metadata/HandleResolver.java
@@ -26,6 +26,7 @@
 import com.facebook.presto.spi.connector.ConnectorPartitioningHandle;
 import com.facebook.presto.spi.connector.ConnectorTransactionHandle;
 import com.facebook.presto.spi.function.FunctionHandle;
+import com.facebook.presto.spi.function.FunctionHandleResolver;
 import com.facebook.presto.split.EmptySplitHandleResolver;
 
 import javax.inject.Inject;

diff --git a/...ain/src/main/java/com/facebook/presto/metadata/StaticFunctionNamespaceHandleResolver.java b/...ain/src/main/java/com/facebook/presto/metadata/StaticFunctionNamespaceHandleResolver.java
@@ -14,6 +14,7 @@
 package com.facebook.presto.metadata;
 
 import com.facebook.presto.spi.function.FunctionHandle;
+import com.facebook.presto.spi.function.FunctionHandleResolver;
 
 public class StaticFunctionNamespaceHandleResolver
         implements FunctionHandleResolver

diff --git a/presto-main/src/main/java/com/facebook/presto/metadata/StaticFunctionNamespaceManager.java b/presto-main/src/main/java/com/facebook/presto/metadata/StaticFunctionNamespaceManager.java
@@ -154,6 +154,7 @@
 import com.facebook.presto.spi.block.BlockEncodingSerde;
 import com.facebook.presto.spi.function.FunctionHandle;
 import com.facebook.presto.spi.function.FunctionMetadata;
+import com.facebook.presto.spi.function.FunctionNamespaceManager;
 import com.facebook.presto.spi.function.OperatorType;
 import com.facebook.presto.spi.function.Signature;
 import com.facebook.presto.spi.function.SqlFunction;

diff --git a/...esto/metadata/FunctionHandleResolver.java → .../spi/function/FunctionHandleResolver.java b/...esto/metadata/FunctionHandleResolver.java → .../spi/function/FunctionHandleResolver.java
@@ -11,9 +11,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package com.facebook.presto.metadata;
-
-import com.facebook.presto.spi.function.FunctionHandle;
+package com.facebook.presto.spi.function;
 
 public interface FunctionHandleResolver
 {

diff --git a/...to/metadata/FunctionNamespaceManager.java → ...pi/function/FunctionNamespaceManager.java b/...to/metadata/FunctionNamespaceManager.java → ...pi/function/FunctionNamespaceManager.java
@@ -11,18 +11,16 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package com.facebook.presto.metadata;
+package com.facebook.presto.spi.function;
 
 import com.facebook.presto.spi.QueryId;
-import com.facebook.presto.spi.function.FunctionHandle;
-import com.facebook.presto.spi.function.FunctionMetadata;
-import com.facebook.presto.spi.function.Signature;
-import com.facebook.presto.spi.function.SqlFunction;
+import com.facebook.presto.spi.api.Experimental;
 import com.facebook.presto.spi.relation.FullyQualifiedName;
 
 import java.util.Collection;
 import java.util.List;
 
+@Experimental
 public interface FunctionNamespaceManager
 {
     void addFunctions(List<? extends SqlFunction> functions);

diff --git a/...data/FunctionNamespaceManagerFactory.java → ...tion/FunctionNamespaceManagerFactory.java b/...data/FunctionNamespaceManagerFactory.java → ...tion/FunctionNamespaceManagerFactory.java
@@ -11,7 +11,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package com.facebook.presto.metadata;
+package com.facebook.presto.spi.function;
 
 public interface FunctionNamespaceManagerFactory
 {