From 7061ae39ccea201cb9ef1050f3c18091679a6e2b Mon Sep 17 00:00:00 2001 From: Oleg Zhurakousky Date: Tue, 20 Oct 2020 15:39:57 +0200 Subject: [PATCH] Add FunctionInspector back to SimpleFunctionRegistry Even though it is deprecated and SimpleFunctionRegistry will throw UnsupportedOperationException in case getRegistration() methid is called we need to re-enable it temporarily to mitigate breaking API changes --- .../function/context/catalog/SimpleFunctionRegistry.java | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/spring-cloud-function-context/src/main/java/org/springframework/cloud/function/context/catalog/SimpleFunctionRegistry.java b/spring-cloud-function-context/src/main/java/org/springframework/cloud/function/context/catalog/SimpleFunctionRegistry.java index d4f0ffdb5..1c625bed1 100644 --- a/spring-cloud-function-context/src/main/java/org/springframework/cloud/function/context/catalog/SimpleFunctionRegistry.java +++ b/spring-cloud-function-context/src/main/java/org/springframework/cloud/function/context/catalog/SimpleFunctionRegistry.java @@ -77,7 +77,7 @@ import org.springframework.util.StringUtils; * @author Oleg Zhurakousky * */ -public class SimpleFunctionRegistry implements FunctionRegistry { +public class SimpleFunctionRegistry implements FunctionRegistry, FunctionInspector { protected Log logger = LogFactory.getLog(this.getClass()); /* * - do we care about FunctionRegistration after it's been registered? What additional value does it bring? @@ -106,6 +106,13 @@ public class SimpleFunctionRegistry implements FunctionRegistry { this.headersField.setAccessible(true); } + @Override + public FunctionRegistration getRegistration(Object function) { + throw new UnsupportedOperationException("FunctionInspector is deprecated. There is no need " + + "to access FunctionRegistration directly since you can interogate the actual " + + "looked-up function (see FunctionInvocationWrapper."); + } + @SuppressWarnings("unchecked") @Override public T lookup(Class type, String functionDefinition, String... expectedOutputMimeTypes) {