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.

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

bug(BigQuery): Throw exception when reload function returns 499 status code #6756

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zajca
Copy link

@zajca zajca commented Nov 1, 2023

When jobTimeoutMs is set to run query and query timeouts WaitTrait causes to wait for result "indefinetly" (Job::MAX_RETRIES).

How to replicate issue:

$client = new BigQueryClient(...);
$client->runQuery($client->query(
    'WITH RECURSIVE counter AS (
              SELECT 1 AS n
              UNION ALL
              SELECT n+1 FROM counter WHERE n < 10000
            )
            
            SELECT 
              a.n AS val1, 
              b.n AS val2
            FROM 
              counter a 
            CROSS JOIN 
              counter b;',
    [
        'configuration' => [
            'jobTimeoutMs' => 10 * 1000,
        ],
    ]
));

Maybe there should be more status codes which should not be retried, but I so far run only into this problem.

@zajca zajca requested review from a team as code owners November 1, 2023 10:43
@zajca zajca changed the title Throw exception when reload function returns 499 status code bug(BigQuery): Throw exception when reload function returns 499 status code Nov 1, 2023
zajca added a commit to keboola/google-bigquery-transformation that referenced this pull request Nov 1, 2023
zajca added a commit to keboola/google-bigquery-transformation that referenced this pull request Nov 1, 2023
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the BigQuery API. label Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigquery Issues related to the BigQuery API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant