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

✨ Data store queries now allow for multiple client ids #4130

Merged

Conversation

dseurotech
Copy link
Contributor

✨ Data store queries now allow for multiple client ids to be queried at once

Data store queries in the rest apis now accept multiple client ids at once, making it easier to fetch data for multiple devices, instead of having to perform multiple queries (one per device).

Copy link

codecov bot commented Nov 8, 2024

Codecov Report

Attention: Patch coverage is 0% with 42 lines in your changes missing coverage. Please review.

Project coverage is 16.75%. Comparing base (1fe8a81) to head (43061c3).
Report is 90 commits behind head on develop.

Files with missing lines Patch % Lines
...a/app/api/resources/v1/resources/DataMessages.java 0.00% 11 Missing ⚠️
...a/app/api/resources/v1/resources/DataChannels.java 0.00% 10 Missing ⚠️
...ua/app/api/resources/v1/resources/DataClients.java 0.00% 10 Missing ⚠️
...ua/app/api/resources/v1/resources/DataMetrics.java 0.00% 10 Missing ⚠️
...p/api/resources/v1/resources/DataMessagesJson.java 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #4130      +/-   ##
=============================================
- Coverage      16.80%   16.75%   -0.05%     
  Complexity        22       22              
=============================================
  Files           2021     2039      +18     
  Lines          52470    52878     +408     
  Branches        4426     4441      +15     
=============================================
+ Hits            8815     8860      +45     
- Misses         43257    43615     +358     
- Partials         398      403       +5     
Files with missing lines Coverage Δ
...tastore/internal/MetricInfoRegistryFacadeImpl.java 0.00% <ø> (ø)
...p/api/resources/v1/resources/DataMessagesJson.java 0.00% <0.00%> (ø)
...a/app/api/resources/v1/resources/DataChannels.java 0.00% <0.00%> (ø)
...ua/app/api/resources/v1/resources/DataClients.java 0.00% <0.00%> (ø)
...ua/app/api/resources/v1/resources/DataMetrics.java 0.00% <0.00%> (ø)
...a/app/api/resources/v1/resources/DataMessages.java 0.00% <0.00%> (ø)

... and 63 files with indirect coverage changes

Signed-off-by: dseurotech <[email protected]>
@Coduz Coduz added the Enhancement This PR/Issue improves an part of Kapua label Dec 18, 2024
@Coduz Coduz merged commit 48922a5 into eclipse:develop Dec 20, 2024
33 of 36 checks passed
@Coduz Coduz deleted the ref-multiple_client_ids_in_data_store_queries branch December 20, 2024 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement This PR/Issue improves an part of Kapua
Projects
Development

Successfully merging this pull request may close these issues.

2 participants