Izeren commented on code in PR #27617:
URL: https://github.com/apache/flink/pull/27617#discussion_r2814083917
##########
flink-core/src/main/java/org/apache/flink/configuration/CoreOptions.java:
##########
@@ -567,4 +567,24 @@ public static ConfigOption<Long>
fileSystemConnectionLimitStreamInactivityTimeou
.longType()
.defaultValue(0L);
}
+
+ /**
+ * Explicitly resolves the conflict between multiple FileSystemFactory
implementations when
+ * multiple jars are loaded for the same schema. Primary use is to allow
configuration based
+ * migration between file systems without the need to build separate
images.
+ *
+ * <p>Config key pattern: {@code fs.<scheme>.priority.<factoryClassName>}
+ */
+ public static ConfigOption<Integer> fileSystemFactoryPriority(String
scheme, String className) {
+ return ConfigOptions.key("fs." + scheme + ".priority." + className)
Review Comment:
It did work as is, when I tested it. I didn't want to mess up with the class
the class reference as it is more error prone when you create config. Unless
you have strong objections, I would prefer to keep it this way
--
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]
