You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
bqjdbc connections are configured via URL parameters.
Tests configure their connections with string literals which is very error prone (what if a parameter is listed twice? what if there are two &?)
We should write a utility class that constructs URLs at a higher level to eliminate these issues. We might consider a ConnectionBuilder to manage the creation of connections.
See:
maybe there should be a ConnectionBuilder that removes all this JDBC Url crap from our own tests ?
bqjdbc
connections are configured via URL parameters.Tests configure their connections with string literals which is very error prone (what if a parameter is listed twice? what if there are two
&
?)We should write a utility class that constructs URLs at a higher level to eliminate these issues. We might consider a
ConnectionBuilder
to manage the creation of connections.See:
Seems like an obvious way to cause bugs.
Originally posted by @fzakaria in #186 (comment)
The text was updated successfully, but these errors were encountered: