-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor connection management in bqjdbc tests #186
Commits on Dec 28, 2023
-
Refactor BQScrollableResultSetFunctionTest connection management.
Use common `ConnectionFromResources.connect` method instead of a bespoke `NewConnection` method. Also, use instance variable/test case scoped connections and result sets instead of static/suite scoped ones. This should help enable test parallelization later. Local testing shows no reliable increase in `BQScrollableResultSetFunctionTest` total runtime (~30 seconds both before and after change).
Configuration menu - View commit details
-
Copy full SHA for 28012fa - Browse repository at this point
Copy the full SHA 28012faView commit details -
Refactor BQForwardOnlyResultSetFunctionTest connection management.
Use common `ConnectionFromResources.connect` method instead of a bespoke `NewConnection` method. Also, use instance variable/test case scoped connections and result sets instead of static/suite scoped ones. This should help enable test parallelization later. Local testing shows no reliable increase in runtime (~30 seconds). This was eased by the introduction of a second BQ connection with legacy SQL disabled (`standardSqlConnection`). Not every test uses both connections, so this is a little wasteful and risks introducing some unreliability, but it isolates test better than continually closing and reopening a static connection.
Configuration menu - View commit details
-
Copy full SHA for 582b761 - Browse repository at this point
Copy the full SHA 582b761View commit details -
Refactor BQResultSetFunctionTest connection management.
Use common `ConnectionFromResources.connect` method instead of a bespoke `NewConnection` method. Also, use instance variable/test case scoped connections and result sets instead of static/suite scoped ones. This should help enable test parallelization later. Local testing shows no reliable increase in runtime (~30 seconds).
Configuration menu - View commit details
-
Copy full SHA for 3479fdc - Browse repository at this point
Copy the full SHA 3479fdcView commit details -
Refactor QueryResultTest connection management.
Use common `ConnectionFromResources.connect` method instead of a bespoke `NewConnection` method. Also, use instance variable/test case scoped connections and result sets instead of static/suite scoped ones. This should help enable test parallelization later. Local testing shows no reliable increase in runtime (~12 seconds). This was eased by the introduction of a second BQ connection with legacy SQL disabled (`standardSqlConnection`). Not every test uses both connections, so this is a little wasteful and risks introducing some unreliability, but it isolates test better than continually closing and reopening a static connection.
Configuration menu - View commit details
-
Copy full SHA for 62f1f38 - Browse repository at this point
Copy the full SHA 62f1f38View commit details -
Refactor QueryResultTest connection management.
Use common `ConnectionFromResources.connect` method instead of a bespoke `NewConnection` method. Also, use instance variable/test case scoped connections and result sets instead of static/suite scoped ones. This should help enable test parallelization later. Local testing shows no reliable increase in total runtime (~6 seconds both before and after change).
Configuration menu - View commit details
-
Copy full SHA for 68f17f5 - Browse repository at this point
Copy the full SHA 68f17f5View commit details -
Remove extra instance variable fixtures.
Now each @before method sets a single instance variable. This is to address code review comments.
Configuration menu - View commit details
-
Copy full SHA for e61041d - Browse repository at this point
Copy the full SHA e61041dView commit details