Skip to content

Conversation

@wolfeeeg
Copy link
Contributor

@wolfeeeg wolfeeeg commented Feb 9, 2023

…ort running queries against extDb

Check List

  • Tests has been run in packages where changes made if available
  • Linter has been run for changed code
  • Tests for the changes have been added if not covered yet
  • Docs have been added / updated if required

Issue Reference this PR resolves

[For example #12]

Description of Changes Made (if issue reference is not provided)

[Description goes here]

@github-actions github-actions bot added the pr:community Contribution from Cube.js community members. label Feb 9, 2023
@wolfeeeg wolfeeeg force-pushed the sql-runner-ext-db branch 2 times, most recently from 8e73387 to 409522b Compare February 9, 2023 13:37
@wolfeeeg wolfeeeg marked this pull request as ready for review February 10, 2023 09:14
@wolfeeeg wolfeeeg requested review from a team as code owners February 10, 2023 09:14
@wolfeeeg wolfeeeg force-pushed the sql-runner-ext-db branch 2 times, most recently from 2af913c to 7019384 Compare February 14, 2023 13:34
@codecov
Copy link

codecov bot commented Feb 14, 2023

Codecov Report

Patch coverage: 40.00% and project coverage change: -25.38 ⚠️

Comparison is base (a9b19fa) 68.14% compared to head (7ce5942) 42.76%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #6142       +/-   ##
===========================================
- Coverage   68.14%   42.76%   -25.38%     
===========================================
  Files         301      153      -148     
  Lines       49738    20238    -29500     
  Branches     5196     5196               
===========================================
- Hits        33892     8655    -25237     
+ Misses      15013    10750     -4263     
  Partials      833      833               
Flag Coverage Δ
cube-backend 42.76% <40.00%> (-0.01%) ⬇️
cubesql ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...orchestrator/src/orchestrator/QueryOrchestrator.ts 54.09% <25.00%> (-1.52%) ⬇️
...-query-orchestrator/src/orchestrator/QueryCache.ts 75.07% <100.00%> (+1.32%) ⬆️

... and 148 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@wolfeeeg wolfeeeg force-pushed the sql-runner-ext-db branch 2 times, most recently from 26676ba to 38344bd Compare February 16, 2023 13:04
@wolfeeeg wolfeeeg changed the title feat(api-gateway, server-core, query-orchestrator): sql-runner - supp… feat(server-core, query-orchestrator): sql-runner - supp… Feb 16, 2023
@buntarb buntarb self-assigned this Mar 6, 2023
Copy link
Contributor

@buntarb buntarb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments.

@wolfeeeg wolfeeeg force-pushed the sql-runner-ext-db branch 2 times, most recently from 9bf22a1 to d062179 Compare March 7, 2023 10:34
@wolfeeeg wolfeeeg requested a review from buntarb March 7, 2023 10:34
Copy link
Contributor

@buntarb buntarb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments.

@wolfeeeg wolfeeeg force-pushed the sql-runner-ext-db branch from d062179 to 0b50e78 Compare March 17, 2023 09:32
@wolfeeeg wolfeeeg force-pushed the sql-runner-ext-db branch from 0b50e78 to 7ce5942 Compare March 17, 2023 13:29
@wolfeeeg wolfeeeg requested a review from buntarb March 17, 2023 13:31
@wolfeeeg wolfeeeg merged commit 32c603d into master Mar 21, 2023
@wolfeeeg wolfeeeg deleted the sql-runner-ext-db branch March 21, 2023 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:community Contribution from Cube.js community members.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants