EDG-255 Reclaim Tag Ownership from Adapters, v2 - phase 3 (Edge)#1457
Conversation
…edge - ProtocolAdapterConfigConverter constructs GenericTag directly from TagEntity and DomainTag fields instead of going through the factory's full tag-map path - ProtocolAdaptersResourceImpl.getTagSchema() calls tagDefinitionClass() directly, removing the null-check fallback to tagConfigurationClass() - Update two test stubs to implement tagDefinitionClass() instead of the removed tagConfigurationClass() - Update HttpProtocolAdapterConfigTest to construct GenericTag directly instead of using the removed convertTagDefinitionObjects()
mschoenert-hivemq
changed the title
Coverage Report
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
A
Tagis an Edge concept. It represents a named, described data point as Edge understands it — with a name, a description, and a lifecycle. ATagDefinitionis the adapter's concern: it is the adapter-specific addressing information needed to locate that data point on a device (e.g. a Modbus register address, an OPC-UA node ID).Edge should have complete control over what a tag is and how it behaves. Protocol adapters should only own what is necessary to address a tag on the device — i.e. the
TagDefinition. Adapters receiveTagobjects and pass them back to Edge, but they must not define or own theTagimplementation.Approach
The migration is done in phases. Phases 1 and 2 are fully compatible — no existing adapter needs to change. Phase 3 is incompatible and cleans up the SDK once all adapters have migrated. Phase 4 seals the interface to enforce the invariant permanently.
Phase 1 — SDK and Edge changes (compatible, no adapter changes required)
Phase 2 — Migrate adapters one at a time (compatible, per-adapter)
Phase 3 — Clean up the SDK (incompatible)
Once all known adapters have migrated: