Skip to content

Commit

Permalink
Assume worst cases
Browse files Browse the repository at this point in the history
  • Loading branch information
a-d committed Oct 17, 2024
1 parent 6333352 commit f3a71be
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion docs/adrs/004-core-class-for-service-integration.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ SAP Cloud SDK flavored requirements:
* Log activity, intermediate results, fallbacks and errors.


## State of core class
## State of Core class

| API | Behavior |
|--------------------------------------------------------------------------------|----------------------------------------------------------------------------------------------------------------------------------------------|
Expand All @@ -84,6 +84,15 @@ Properties:
* Immutable objects.
* Custom method overloads on (sub-classes of) `AiCoreService` are propagated down to the client.

Consideration:
* Always assume worst-case:
* User instantiates a constant `AiCoreService` in a static context.
* Custom Service Binding may be supplied by 3nd party library.
Race condition for resolving service bindings may lead to inconsistent behavior.
* User switches resource group at runtime.
* User switches destination at runtime.
* User switches deployment at runtime.

Pro:
- No redundant HTTP traffic.
- Slim API contract, not many public methods.
Expand Down

0 comments on commit f3a71be

Please sign in to comment.