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

Completed db cleanup #192

Merged
merged 5 commits into from
Sep 27, 2024
Merged

Completed db cleanup #192

merged 5 commits into from
Sep 27, 2024

Conversation

jo-asplin-met-no
Copy link
Contributor

@jo-asplin-met-no jo-asplin-met-no commented Sep 24, 2024

This change completes the periodic database cleanup by also deleting unreferred rows in tables time_series and geo_point. Calls to the cleanup function are also now triggered from a single goroutine that runs independently of calls to PutObservations().

This change completes the periodic database cleanup by also deleting
unreferred rows in tables time_series and geo_point. The cleanup is
also now triggered from a single goroutine that runs independently
of calls to PutObservations().
Copy link

github-actions bot commented Sep 24, 2024

API Unit Test Coverage Report
FileStmtsMissCoverMissing
\_\_init\_\_.py00100% 
datastore_pb2.py614821%34–81
datastore_pb2_grpc.py542750%15–16, 19, 57–72, 105–107, 112–114, 119–121, 126–128, 132–157, 195, 222, 249, 276
export_metrics.py100100% 
grpc_getter.py201145%15–19, 23–26, 30–32, 36–38
locustfile.py15150%1–31
main.py43784%45, 50, 60, 70–71, 81–82
metadata_endpoints.py632954%45–54, 58, 85, 100–216, 220
response_classes.py50100% 
utilities.py1815669%20, 38, 45, 67–70, 78–89, 94–101, 121, 125, 127, 154, 157, 166, 184–185, 189, 205–210, 215, 219–220, 224–226, 232–240, 250–253, 259–260, 272–276, 299, 304, 317
custom_geo_json
   edr_feature_collection.py60100% 
formatters
   \_\_init\_\_.py110100% 
   covjson.py57198%88
   geojson.py17288%21, 46
openapi
   custom_dimension_examples.py40100% 
   edr_query_parameter_descriptions.py110100% 
   openapi_examples.py130100% 
routers
   \_\_init\_\_.py00100% 
   edr.py102496%350–351, 441–442
   feature.py471960%99–132, 148–153, 159–181
TOTAL72021970% 

API Unit Test Coverage Summary

Tests Skipped Failures Errors Time
30 0 💤 0 ❌ 0 🔥 1.948s ⏱️

Copy link
Contributor

@lukas-phaf lukas-phaf left a comment

Choose a reason for hiding this comment

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

Much cleaner solution! ;-)

And what would a go program be without a channel...

@jo-asplin-met-no jo-asplin-met-no merged commit 5aabf54 into main Sep 27, 2024
8 checks passed
@jo-asplin-met-no jo-asplin-met-no deleted the issue_167_complete_cleanup_new branch September 27, 2024 07:13
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.

2 participants