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

Cache Results of the Exists CQL Expression #1796

Merged
merged 1 commit into from
Jun 26, 2024
Merged

Conversation

alexanderkiel
Copy link
Member

Closes: #1051

@alexanderkiel alexanderkiel self-assigned this Jun 12, 2024
@alexanderkiel alexanderkiel added this to the v0.28.0 milestone Jun 12, 2024
@alexanderkiel alexanderkiel added enhancement New feature or request module:cql labels Jun 12, 2024
@alexanderkiel alexanderkiel force-pushed the cql-cache-3 branch 3 times, most recently from 6ed6a7a to 6e7584e Compare June 13, 2024 11:34
Copy link

codecov bot commented Jun 13, 2024

Codecov Report

Attention: Patch coverage is 80.92255% with 335 lines in your changes missing coverage. Please review.

Project coverage is 94.74%. Comparing base (48942a8) to head (fa18fd8).
Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1796      +/-   ##
===========================================
- Coverage    96.20%   94.74%   -1.46%     
===========================================
  Files          271      282      +11     
  Lines        17353    18412    +1059     
  Branches       347      428      +81     
===========================================
+ Hits         16694    17445     +751     
- Misses         312      539     +227     
- Partials       347      428      +81     
Files Coverage Δ
modules/cql/src/blaze/elm/compiler.clj 100.00% <100.00%> (ø)
.../cql/src/blaze/elm/compiler/clinical_operators.clj 100.00% <100.00%> (ø)
...ql/src/blaze/elm/compiler/library/resolve_refs.clj 100.00% <100.00%> (ø)
...ules/cql/src/blaze/elm/compiler/type_operators.clj 89.35% <100.00%> (+1.78%) ⬆️
modules/cql/src/blaze/elm/date_time.clj 93.23% <ø> (ø)
modules/cql/src/blaze/elm/expression.clj 100.00% <ø> (ø)
...ql/src/blaze/elm/expression/cache/bloom_filter.clj 100.00% <100.00%> (ø)
.../cql/src/blaze/elm/expression/cache/codec/by_t.clj 100.00% <100.00%> (ø)
.../cql/src/blaze/elm/expression/cache/codec/form.clj 100.00% <100.00%> (ø)
...s/cql/src/blaze/elm/expression/cache/protocols.clj 100.00% <100.00%> (ø)
... and 46 more

... and 3 files with indirect coverage changes

@alexanderkiel alexanderkiel force-pushed the cql-cache-3 branch 8 times, most recently from c3c7c08 to 41d6e82 Compare June 18, 2024 16:28
@alexanderkiel alexanderkiel force-pushed the cql-cache-3 branch 10 times, most recently from 17b07f4 to 8717e3b Compare June 25, 2024 17:22
@alexanderkiel alexanderkiel merged commit 7afce4b into develop Jun 26, 2024
165 of 167 checks passed
@alexanderkiel alexanderkiel deleted the cql-cache-3 branch June 26, 2024 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request module:cql
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant