Adjustments to Ditto java client after elimination of concierge service in Ditto#190
Merged
Yannic92 merged 14 commits intoeclipse-ditto:masterfrom Aug 4, 2022
Merged
Conversation
* removing SignalInformationPoint from ditto-interna-models-signal (as it pulled in several entity modules); moved funtionality of that interface to Command, Signal, etc. interfaces * moving HeaderTranslator from ditto-protocol to ditto-base-model * moving LikeHelper from ditto-rql-query to ditto-base-model * moved ConciergeForwarderActor and ShardRegions to ditto-edge-api * moved PolicyEnforcer from ditto-internal-utils-cache-loaders to ditto-policies-enforcement * created DittoSystemProperties inn ditto-base-model for commonly used system properties in order to avoid dependencies to models where not necessary * moved ThingCommandEnforcement, LiveSignalEnforcement, etc. from ditto-policies-enforcement to ditto-things-service where they should be located in the end Signed-off-by: Thomas Jaeckle <[email protected]>
Signed-off-by: Thomas Jaeckle <[email protected]>
…le "gateway-api" * added first "pass through" implementation of ConnectionEnforcerActor and ConnectivityCommandEnforcement * made CleanupCommand a SudoCommand * moved Signal.hasTypePrefix to WithType.hasTypePrefix and added constants for commonly known commands in WithType * moved exceptions around and fixed DittoRuntimeExceptions with missing prefixes, e.g. added ProtocolAdapterException for ditto-protocol-adapter Signed-off-by: Thomas Jaeckle <[email protected]>
…mespace" * to use when creating new things/policies without specifying an ID Signed-off-by: Thomas Jaeckle <[email protected]>
Signed-off-by: Thomas Jaeckle <[email protected]>
Signed-off-by: Thomas Jaeckle <[email protected]>
* This adds unnecessary complexity/dependency. It's required that client and ditto instance have same config, otherwise things went strange. Leave responsibility for resolving the default namespace to the service. Signed-off-by: Yannic Klem <[email protected]>
Signed-off-by: Thomas Jaeckle <[email protected]>
…onal headers * fixed ExecutePiggybackCommand serialization / deserialization at DevOpsRoute by including the sent "headers" * made AddConnectionLogEntry a SudoCommand and moved it accordingly * pulled out SudoAddConnectionLogEntryStrategy as separate strategy * got rid of explicit knowledge of "connectivityShardRegionProxy" in gateway, instead delegate connectivity commands via edge command forwarder * moved "ThingsOutOfSync" event to search api module * made "StreamThings" search command a SudoCommand Signed-off-by: Thomas Jaeckle <[email protected]>
Signed-off-by: Thomas Jaeckle <[email protected]>
…picmp excludes accordingly Signed-off-by: Thomas Jaeckle <[email protected]>
Signed-off-by: Thomas Jaeckle <[email protected]>
This reverts commit cc23bdb.
…usted japicmp excludes accordingly" This reverts commit 25fbb83.
Yannic92
approved these changes
Aug 4, 2022
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
2 participants
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.
Necessary changes following up Ditto PR eclipse-ditto/ditto#1416