zentol commented on a change in pull request #16345:
URL: https://github.com/apache/flink/pull/16345#discussion_r662550583
##########
File path:
flink-rpc/flink-rpc-core/src/main/java/org/apache/flink/runtime/rpc/RpcSystem.java
##########
@@ -74,8 +87,39 @@ RpcServiceBuilder withExecutorConfiguration(
* @return loaded RpcSystem
*/
static RpcSystem load() {
- final ClassLoader classLoader = RpcSystem.class.getClassLoader();
- return ServiceLoader.load(RpcSystem.class,
classLoader).iterator().next();
+ return load(new Configuration());
+ }
+
+ /**
+ * Loads the RpcSystem.
+ *
+ * @return loaded RpcSystem
+ */
+ static RpcSystem load(Configuration config) {
+ try {
+ final ClassLoader classLoader = RpcSystem.class.getClassLoader();
+
+ final String tmpDirectory =
ConfigurationUtils.parseTempDirectories(config)[0];
+ final Path tempFile =
+ Files.createFile(
+ Paths.get(tmpDirectory, UUID.randomUUID() +
"_flink-rpc-akka.jar"));
Review comment:
It is a bit annoying that something from flink-rpc-core is now so
hard-wired to flink-rpc-akka, so maybe we should the loading to flink-runtime.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
