Github user 10110346 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20802#discussion_r174332968
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -975,6 +975,8 @@ private[spark] object Utils extends Logging {
def checkHost(host: String) {
assert(host != null && host.indexOf(':') == -1, s"Expected hostname
(not IP) but got $host")
+ assert(host.matches("^[a-zA-Z0-9-.]+$"),
--- End diff --
Yeah, it looks not very reasonable for this check, there is no good way to
do it.
I might have to close this PR
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
