yunfengzhou-hub commented on code in PR #26652:
URL: https://github.com/apache/flink/pull/26652#discussion_r2148897320
##########
flink-core/src/main/java/org/apache/flink/configuration/GlobalConfiguration.java:
##########
@@ -52,6 +52,7 @@ public final class GlobalConfiguration {
"secret",
"fs.azure.account.key",
"apikey",
+ "api-key",
Review Comment:
According to previous comments with @fsk119, existing practices about Model
Function have used `api-key` instead of `apikey`, so the same convention is
also followed here.
https://trino.io/docs/current/functions/ai.html
As for `apikey`, it seems that it is used in `DatadogHttpReporterFactory`, a
custom metric reporter. Given that both these configurations are defined by
Flink plugins rather than Flink core, it should be OK that they do not follow
the same rule.
--
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]
