KarmaGYZ commented on a change in pull request #11854:
URL: https://github.com/apache/flink/pull/11854#discussion_r414340925



##########
File path: 
flink-core/src/main/java/org/apache/flink/api/common/functions/util/AbstractRuntimeUDFContext.java
##########
@@ -175,6 +177,11 @@ public DistributedCache getDistributedCache() {
                return this.distributedCache;
        }
 
+       @Override
+       public <T extends ExternalResourceInfo> Set<T> 
getExternalResourceInfos(String resourceName, Class<T> externalResourceType) {
+               throw new UnsupportedOperationException("Do not support 
external resource in current environment");
+       }
+

Review comment:
       The `RuntimeUDFContext` and its subclass is used for [Collection 
Environment](https://ci.apache.org/projects/flink/flink-docs-master/dev/local_execution.html#collection-environment).
 As the docs say:
   
   > Typical use-cases for this mode are automated tests, debugging and code 
re-use.
   
   This mode will not start the full Flink runtime, the task executor will not 
be started. Since currently the external resource is a TM level component. From 
my side, it probably not make sense to support it in this environment.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to