Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: adds query optimizer statistics support #385

Merged
merged 9 commits into from Jun 5, 2021
Merged

feat: adds query optimizer statistics support #385

merged 9 commits into from Jun 5, 2021

Commits on Jun 5, 2021

  1. feat: adds optimizer_statistics_package option 

    Adds the possibility to set the optimizer statistics package when
executing queries. This option can be set in different levels as follows:
1. Through statement hints
2. Through query level configuration
3. Through an environment variable
4. Through application level configuration

If more than one package is set (in different levels) the precedence is
from 1 to 4 (1 has top priority).
    @thiagotnunes
    thiagotnunes committed Jun 5, 2021
    Copy the full SHA
    a12b81c View commit details
    Browse the repository at this point in the history

  2. test: adds integration tests 

    Adds integration tests for setting the query options optimizer
statistics package.
    @thiagotnunes
    thiagotnunes committed Jun 5, 2021
    Copy the full SHA
    c16a2dc View commit details
    Browse the repository at this point in the history

  3. fix: addresses PR comments 

    Fix missing value in the documentation of the optimizer statistics
package for the connection class.
    @thiagotnunes
    thiagotnunes committed Jun 5, 2021
    Copy the full SHA
    2173166 View commit details
    Browse the repository at this point in the history

  4. fix: adds tests for invalid stats packages 

    Adds tests for invalid statistics packages (whitespace only ones).
    @thiagotnunes
    thiagotnunes committed Jun 5, 2021
    Copy the full SHA
    5a25b97 View commit details
    Browse the repository at this point in the history

  5. fix: adds integration tests for query options 

    Adds an integration test to run the sql script with several expectations
for the query options. These are tests for the optimizer version and
optimizer statistics package.
    @thiagotnunes
    thiagotnunes committed Jun 5, 2021
    Copy the full SHA
    6a48abe View commit details
    Browse the repository at this point in the history

  6. fix: formatting of ClientSideStatements.json 

    This file is using a mix of tabs and spaces. For now I have formatted as
the other lines, so that the diff is clear. In a further PR I will
reformat the whole file to use only spaces.
    @thiagotnunes
    thiagotnunes committed Jun 5, 2021
    Copy the full SHA
    954dfc8 View commit details
    Browse the repository at this point in the history

  7. Merge branch 'master' of github.com:thiagotnunes/java-spanner into ad… 

    …ds-query-optimizer-statistics-support
    @thiagotnunes
    thiagotnunes committed Jun 5, 2021
    Copy the full SHA
    8cf5f10 View commit details
    Browse the repository at this point in the history

  8. tests: fix Connection test 

    Fixes connection test when using environment variables for retrieving
configurations. This only works when a first connection is created, so
we moved this specific test to its own subclass.
    @thiagotnunes
    thiagotnunes committed Jun 5, 2021
    Copy the full SHA
    5d7350c View commit details
    Browse the repository at this point in the history

  9. fix: fix clirr checks 

    Provides default interface implementations and fixes clirr checks
    @thiagotnunes
    thiagotnunes committed Jun 5, 2021
    Copy the full SHA
    c3e365a View commit details
    Browse the repository at this point in the history