Remove threadid from connectionkey

Author: quinnj

src/Connections.jl

@@ -95,12 +95,12 @@ end
 Used for "hashing" a Connection object on just the key properties necessary for determining
 connection re-useability. That is, when a new request calls `newconnection`, we take the
 request parameters of host and port, and if ssl verification is required, if keepalive is enabled,
-if an existing Connection was already created with the exact, and the current thread id.
+and if an existing Connection was already created with the exact.
 same parameters, we can re-use it (as long as it's not already being used, obviously).
 """
-connectionkey(x::Connection) = (x.host, x.port, x.require_ssl_verification, x.keepalive, x.clientconnection, Threads.threadid())
+connectionkey(x::Connection) = (x.host, x.port, x.require_ssl_verification, x.keepalive, x.clientconnection)
 
-const ConnectionKeyType = Tuple{AbstractString, AbstractString, Bool, Bool, Bool, Int}
+const ConnectionKeyType = Tuple{AbstractString, AbstractString, Bool, Bool, Bool}
 
 Connection(host::AbstractString, port::AbstractString,
            idle_timeout::Int,
@@ -445,7 +445,7 @@ function newconnection(::Type{T},
     connection_limit_warning(connection_limit)
     return acquire(
             getpool(pool, T),
-            (host, port, require_ssl_verification, keepalive, true, Threads.threadid());
+            (host, port, require_ssl_verification, keepalive, true);
             forcenew=forcenew,
             isvalid=c->connection_isvalid(c, Int(idle_timeout))) do
         Connection(host, port,