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

refactor(dup): port queries to xandra #838

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

noaccOS
Copy link
Contributor

@noaccOS noaccOS commented Aug 29, 2023

Ports all the existing queries to xandra, with a little refactor where deemed necessary (no, no "simple flow" rework yet).
Removes all references to CQEx.

test(dup): remove a few warnings
chore(dup): update xandra 0.13 -> 0.17

Remove some unused aliases and variables

Signed-off-by: Francesco Noacco <[email protected]>
Ports all the existing queries to xandra, with a little refactor where deemed necessary (no, no `"simple flow"` rework _yet_).
Removes all references to CQEx.

Signed-off-by: Francesco Noacco <[email protected]>
@codecov
Copy link

codecov bot commented Aug 31, 2023

Codecov Report

Merging #838 (d498777) into master (e0fd426) will increase coverage by 14.02%.
Report is 12 commits behind head on master.
The diff coverage is 0.00%.

❗ Current head d498777 differs from pull request most recent head 33493ad. Consider uploading reports for the commit 33493ad to get more accurate results

@@             Coverage Diff             @@
##           master     #838       +/-   ##
===========================================
+ Coverage   67.42%   81.45%   +14.02%     
===========================================
  Files         264       70      -194     
  Lines        6429     1709     -4720     
===========================================
- Hits         4335     1392     -2943     
+ Misses       2094      317     -1777     
Files Changed Coverage Δ
...ne_api/lib/astarte_appengine_api/device/queries.ex 80.92% <0.00%> (ø)

... and 196 files with indirect coverage changes

@Annopaolo Annopaolo mentioned this pull request Sep 5, 2023
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.

1 participant