Good catch. Maybe a try-finally where we just `Dispose()` all 
`createdConnections` in the `finally` block?

I think if a connection establishment fails then we should just let the 
exception be raised to the user as we can't really handle that.
(Maybe it makes sense later to add a retry here, but we should probably have a 
logger for cases like that so users can notice such problems.)

Or what do you think?

[ Full content available at: https://github.com/apache/tinkerpop/pull/1077 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to