dmvk commented on a change in pull request #18522:
URL: https://github.com/apache/flink/pull/18522#discussion_r796792992
##########
File path:
flink-core/src/main/java/org/apache/flink/configuration/BlobServerOptions.java
##########
@@ -33,7 +33,7 @@
key("blob.storage.directory")
.noDefaultValue()
Review comment:
Sorry, I've missed this one.
I'm not sure how to enforce it. We could have it return `URI` type, which
could be a motivation for using it to a certain degree (the same logic kind of
applies to why not have `int` values stored as `string`).
Anyway, it was just an idea, no need to block the PR on this.
--
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]
