Skip to content

Conversation

@ovr
Copy link
Member

@ovr ovr commented Jan 23, 2023

Hello!

refs #5042

Thanks

@ovr ovr requested review from a team as code owners January 23, 2023 19:56
@ovr ovr force-pushed the cubestore-driver-persistent-query branch from 697ecd3 to 08076d2 Compare January 23, 2023 20:08
@codecov
Copy link

codecov bot commented Jan 23, 2023

Codecov Report

Base: 41.41% // Head: 41.57% // Increases project coverage by +0.16% 🎉

Coverage data is based on head (1af10e3) compared to base (c5b6702).
Patch coverage: 73.17% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6046      +/-   ##
==========================================
+ Coverage   41.41%   41.57%   +0.16%     
==========================================
  Files         151      152       +1     
  Lines       20086    20091       +5     
  Branches     5242     5242              
==========================================
+ Hits         8318     8353      +35     
+ Misses      11451    11428      -23     
+ Partials      317      310       -7     
Flag Coverage Δ
cube-backend 41.57% <73.17%> (+0.16%) ⬆️

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

Impacted Files Coverage Δ
...-query-orchestrator/src/orchestrator/QueryCache.ts 66.33% <0.00%> (ø)
packages/cubejs-server-core/src/core/server.ts 66.01% <25.00%> (-0.38%) ⬇️
packages/cubejs-backend-shared/src/process.ts 75.00% <75.00%> (ø)
...-query-orchestrator/src/orchestrator/QueryQueue.js 71.72% <81.48%> (+9.45%) ⬆️
packages/cubejs-backend-shared/src/index.ts 100.00% <100.00%> (ø)
...y-orchestrator/src/orchestrator/PreAggregations.ts 78.90% <100.00%> (-0.03%) ⬇️
...ubejs-query-orchestrator/src/orchestrator/utils.ts 86.41% <100.00%> (-0.49%) ⬇️
...es/cubejs-schema-compiler/src/adapter/BaseQuery.js 70.39% <0.00%> (-0.39%) ⬇️
...query-orchestrator/src/orchestrator/QueryStream.ts 30.30% <0.00%> (+21.21%) ⬆️

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 at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ovr ovr force-pushed the cubestore-driver-persistent-query branch 2 times, most recently from fcd9322 to 145a611 Compare January 25, 2023 16:00
@ovr ovr force-pushed the cubestore-driver-persistent-query branch from 145a611 to e322e93 Compare January 25, 2023 16:36
// closing stream
if (query?.queryHandler === 'stream') {
const stream = this.getQueryStream(this.redisHash(queryKey));
const stream = this.getQueryStream(queryKey);
Copy link
Member Author

Choose a reason for hiding this comment

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

image

@ovr ovr force-pushed the cubestore-driver-persistent-query branch from 5c450ff to f84c86f Compare January 25, 2023 17:36
@ovr ovr force-pushed the cubestore-driver-persistent-query branch from e196e92 to e8b565f Compare January 25, 2023 17:47
case 'stream':
const target = this.getQueryStream(queryKey);
await this.queryTimeout(this.queryHandlers.stream(query.query, target));
await this.queryTimeout(
Copy link
Member Author

Choose a reason for hiding this comment

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

changed due to linter.

@ovr ovr requested a review from a team as a code owner January 25, 2023 18:57
@ovr ovr force-pushed the cubestore-driver-persistent-query branch from d2f5739 to 1bd87af Compare January 25, 2023 22:16
@ovr ovr force-pushed the cubestore-driver-persistent-query branch from 1bd87af to 7e64285 Compare January 25, 2023 22:27
@ovr ovr force-pushed the cubestore-driver-persistent-query branch from 160f6fb to c139999 Compare January 25, 2023 23:02
@ovr ovr merged commit 5b12ec8 into master Jan 26, 2023
@ovr ovr deleted the cubestore-driver-persistent-query branch January 26, 2023 12:00
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.

3 participants