jerrypeng commented on pull request #13514: URL: https://github.com/apache/flink/pull/13514#issuecomment-709519491
@zentol If we change the MetricsQueryService to not exclude/replace any special characters, there might be backwards compatibility issues. For example, if a user has a service that calls the REST endpoint in Flink for metrics, this change will potentially break those use cases as now it is up to the user to escape/replace special characters. Since MetricsQueryService already establishes the pattern of replacing special characters, I am in favor of implicitly replacing all special characters there. ---------------------------------------------------------------- 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: [email protected]
