Skip to content

fix: Fetch summary SQL state 54000#487

Merged
roneli merged 1 commit intocloudquery:mainfrom
roneli:fix/fetch_summary_insert
Feb 13, 2022
Merged

fix: Fetch summary SQL state 54000#487
roneli merged 1 commit intocloudquery:mainfrom
roneli:fix/fetch_summary_insert

Conversation

@roneli
Copy link
Copy Markdown
Contributor

@roneli roneli commented Feb 13, 2022

if we have many diagnostics in the fetch summary it will cause an OOM memory insert failure, we remove the diagnostics from the summary, as they are logged regardless.

if we have many diagnostics in the fetch summary it will cause an OOM memory insert failure, we remove the diagnostics from the summary, as they are logged regardless.
@roneli roneli requested a review from disq February 13, 2022 15:13
@github-actions github-actions bot added the fix label Feb 13, 2022
@roneli roneli merged commit 9e6dbd9 into cloudquery:main Feb 13, 2022
@roneli roneli deleted the fix/fetch_summary_insert branch February 13, 2022 18:38
TinLe pushed a commit to TinLe/cloudquery that referenced this pull request Feb 22, 2022
if we have many diagnostics in the fetch summary it will cause an OOM memory insert failure, we remove the diagnostics from the summary, as they are logged regardless.
TinLe pushed a commit to TinLe/cloudquery that referenced this pull request Feb 22, 2022
* upstream/main:
  feat: Policy Snapshot Testing (cloudquery#449)
  fix: Remove enable_partial_fetch and support for migrationless providers (cloudquery#495)
  fix: Upgrade schema in history mode (cloudquery#494)
  fix: Don't report unmanaged provider diags to sentry (cloudquery#492)
  feat: Modules v2 support (cloudquery#489)
  chore: Synced local '.github/dependabot.yml' with remote 'dependabot.yml' (cloudquery#490)
  chore: Remove '--debug' flag from issue template (cloudquery#488)
  fix: Don't return an error if encountering an empty subpolicy (cloudquery#486)
  fix: Fetch summary SQL state 54000 (cloudquery#487)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants