Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Alex Aperis <[email protected]>
  • Loading branch information
blankdots and aaperis authored Nov 30, 2023
1 parent f1fbf8a commit 2fe2d67
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions docs/connection.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ components to send and receive messages, which are queued, not lost, and
resent on network failure or connection problems.

The RabbitMQ message brokers of each SDA instance are the **only**
components with the necessary credentials to connect to CentralEGA
components with the necessary credentials to connect to `CentralEGA`
message broker.

We call `CEGAMQ` and `LocalMQ` (Local Message Broker, sometimes know as `sda-mq`),
Expand Down Expand Up @@ -136,7 +136,7 @@ and the shovel mechanism in their local broker.
instance, on the given `vhost`.

The exchanges and routing keys will be the same as all the other
FederatedEGA instances, since the clustering is done per `vhost`.
`FederatedEGA` instances, since the clustering is done per `vhost`.

### Message Format

Expand Down
2 changes: 1 addition & 1 deletion docs/structure.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ In a Federated setup, the `FederatedEGA` archive node setup locally need to exch

In a stand-alone setup, the deployed service has less remote synchronisation to worry about, but on the other hand more components might be required (e.g ([orchestrator](services/orchestrator))) to also handle annotations/meta-data locally, as well as to deal with identifiers etc.

The NeIC SDA is targeting both types of setup, to open up the use of the re-use of components to more use cases than initially envisioned.
The NeIC SDA is targeting both types of setup but also to allow for the possibility to re-use components in more use cases than initially envisioned.


### Container deployment options
Expand Down

0 comments on commit 2fe2d67

Please sign in to comment.