Skip to content

Conversation

@FelixTing
Copy link
Member

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md

PR Checklist

Please check if your PR fulfills the following requirements:

  • I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
  • I am not introducing a new dependency (add notes below if you are)
  • I have added unit tests for the new feature or bug fix (if not, why?)
  • I have fully tested (add details below) this the new feature or bug fix (if not, why?)
  • I have opened a PR for the related docs change (if not, why?)

weichou1229 and others added 8 commits October 15, 2025 14:31
Update CoreData API doc and fix format

Signed-off-by: Bruce Huang <[email protected]>
The sample for POST new device specifies string for both Port and
UnitID, which are incorrect and should be corrected to use numbers.

Signed-off-by: Jude Hung <[email protected]>
Query events without total count to prevent large dataset query and count.

Signed-off-by: Bruce Huang <[email protected]>
weichou1229 and others added 12 commits October 15, 2025 14:36
…ndry#5164)

CoreCommand GET /device/all API check whether device profile is empty, if empty skip the profile query for the specific device.

Close edgexfoundry#5163

Signed-off-by: Bruce Huang <[email protected]>
…oundry#5174)

Fix CoreCommand GET /device/all API incorrect totalCount

Close edgexfoundry#5172

Signed-off-by: Bruce Huang <[email protected]>
converts a path to the format required for go embed by using '/' as the separator.
Close edgexfoundry#5150

Signed-off-by: Cloud Tsai <[email protected]>
…dgexfoundry#5177)

CoreData should not return the events when enabling EventPurge and removing device from metadata. This issue occurs when user want to purge large amount of event data.

- Adds mark_deleted field to CoreData deviceInfo table
- Marks deviceInfo as deletable before deleting events adn readings
- Add mark_deleted

Close edgexfoundry#5175

Signed-off-by: Bruce Huang <[email protected]>
…foundry#5189)

CoreData apply default retention policy to events not coming from auto events

Close edgexfoundry#5188

Signed-off-by: Bruce Huang <[email protected]>
Add the event_id index on reading table to speed up the query.

Close edgexfoundry#5176

Signed-off-by: Bruce Huang <[email protected]>
1. go:S117
2. go:S1186

Signed-off-by: Jude Hung <[email protected]>
@sonarqubecloud
Copy link

@sonarqubecloud
Copy link

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 32.06107% with 89 lines in your changes missing coverage. Please review.
✅ Project coverage is 26.74%. Comparing base (78a2866) to head (738e5d8).

Files with missing lines Patch % Lines
internal/pkg/infrastructure/postgres/sql.go 0.00% 23 Missing ⚠️
internal/core/data/application/reading.go 53.65% 18 Missing and 1 partial ⚠️
internal/core/data/application/event.go 18.18% 15 Missing and 3 partials ⚠️
internal/pkg/infrastructure/postgres/event.go 0.00% 6 Missing ⚠️
internal/pkg/infrastructure/postgres/device.go 0.00% 5 Missing ⚠️
internal/pkg/infrastructure/postgres/deviceinfo.go 0.00% 5 Missing ⚠️
internal/pkg/infrastructure/redis/device.go 0.00% 5 Missing ⚠️
internal/pkg/cache/device.go 0.00% 4 Missing ⚠️
internal/pkg/db/postgres/utils.go 0.00% 3 Missing ⚠️
internal/pkg/infrastructure/postgres/reading.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           odessa    #5285      +/-   ##
==========================================
+ Coverage   26.59%   26.74%   +0.15%     
==========================================
  Files         277      277              
  Lines       19927    19994      +67     
==========================================
+ Hits         5299     5348      +49     
- Misses      13373    13387      +14     
- Partials     1255     1259       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@FelixTing FelixTing requested a review from judehung November 18, 2025 07:19
Copy link
Member

@judehung judehung left a comment

Choose a reason for hiding this comment

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

LGTM

@judehung judehung merged commit e1cefd0 into edgexfoundry:odessa Nov 18, 2025
5 checks passed
@FelixTing FelixTing deleted the pick-fix-from-main branch November 18, 2025 10:35
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.

7 participants