Skip to content

Commit

Permalink
CORE-15756 Segregated and Granular Corda Workers (#1227)
Browse files Browse the repository at this point in the history
Updating topic permissions for the new workers: `PersistenceWorker`, `UniquenessWorker`, and `FlowMapperWorker` which shadow some topic permissions from the `DBWorker` and `FlowWorker`
  • Loading branch information
mbrkic-r3 authored Aug 31, 2023
1 parent 805f96b commit 7ea0236
Show file tree
Hide file tree
Showing 12 changed files with 40 additions and 10 deletions.
2 changes: 1 addition & 1 deletion data/avro-schema/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
## Avro Code Source Files

All Arvo definitions should go under `src/main/resources/avro`. While technically
All Avro definitions should go under `src/main/resources/avro`. While technically
not necessary, it would also be good to put them under the same directory structure as
the package (or `namespace` in Avro-speak) that the record will belong.

Expand Down
2 changes: 1 addition & 1 deletion data/topic-schema/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ The schemas for each topic a virtual node uses will be defined here.
These schemas should be updated here alongside their implementation in an effort to keep this document up-to-date.

### Kafka Auto-topic creation
**We must ensure to avoid setting auto-creation of topics is Kafka!**
**We must ensure we do not enable auto-creation of topics in Kafka!**

This can lead to unnoticed errors in production. We should adhere to the ethos that if a topic
we expect doesn't exist then that signifies a setup problem and should immediately cause an error.
Original file line number Diff line number Diff line change
Expand Up @@ -5,18 +5,24 @@ topics:
- crypto
- db
- flow
- flowMapper
- verification
- membership
- gateway
- link-manager
- persistence
- rest
- uniqueness
producers:
- crypto
- db
- flow
- flowMapper
- verification
- membership
- gateway
- link-manager
- persistence
- rest
- uniqueness
config:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,13 @@ topics:
- crypto
- db
- flow
- flowMapper
- verification
- membership
- gateway
- link-manager
- rest
- persistence
- uniqueness
producers:
- db
Expand Down
11 changes: 10 additions & 1 deletion data/topic-schema/src/main/resources/net/corda/schema/Flow.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,9 @@ topics:
- db
- crypto
- flow
- flowMapper
- verification
- persistence
- uniqueness
config:
FlowEventStateTopic:
Expand All @@ -28,21 +31,26 @@ topics:
- flow
producers:
- flow
- verification
config:
FlowMapperEventTopic:
name: flow.mapper.event
consumers:
- flow
- flowMapper
producers:
- flow
- flowMapper
- rest
config:
FlowMapperEventStateTopic:
name: flow.mapper.event.state
consumers:
- flow
- flowMapper
producers:
- flow
- flowMapper
config:
cleanup.policy: compact
segment.ms: 600000
Expand All @@ -54,8 +62,9 @@ topics:
name: flow.mapper.event.dlq
consumers:
- flow
- flowMapper
producers:
- flow
- flowMapper
config:
FlowStatusTopic:
name: flow.status
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ topics:
consumers:
- db
- flow
- flowMapper
- link-manager
- membership
- rest
Expand Down Expand Up @@ -62,6 +63,7 @@ topics:
- membership
- rest
- flow
- flowMapper
- db
producers:
- membership
Expand Down Expand Up @@ -91,6 +93,7 @@ topics:
consumers:
- db
- flow
- flowMapper
producers:
- link-manager
- membership
Expand All @@ -104,6 +107,7 @@ topics:
- membership
- rest
- flow
- flowMapper
- db
producers:
- db
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ topics:
P2PInTopic:
name: p2p.in
consumers:
- flow
- flowMapper
- membership
producers:
- link-manager
Expand All @@ -12,7 +12,7 @@ topics:
consumers:
- link-manager
producers:
- flow
- flowMapper
- membership
config:
P2POutDlqTopic:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,14 @@ topics:
PersistenceEntityProcessorTopic:
name: persistence.entity.processor
consumers:
- db
- persistence
producers:
- flow
config:
PersistenceLedgerProcessorTopic:
name: persistence.ledger.processor
consumers:
- db
- persistence
producers:
- flow
config:
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ topics:
producers:
- flow
- db
- persistence
config:
ServicesTokenEventStateTopic:
name: services.token.event.state
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
topics:
PersistenceEntityProcessorTopic:
VerificationProcessorTopic:
name: verification.ledger.processor
consumers:
- flow
- verification
producers:
- flow
config:
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,11 @@ topics:
consumers:
- db
- flow
- flowMapper
- verification
- membership
- link-manager
- persistence
- rest
producers:
- db
Expand Down Expand Up @@ -34,7 +37,9 @@ topics:
name: cpk.file
consumers:
- flow
- verification
- db
- persistence
producers:
- db
config:
Expand Down Expand Up @@ -64,8 +69,10 @@ topics:
- crypto
- db
- flow
- flowMapper
- membership
- link-manager
- persistence
- rest
producers:
- db
Expand Down
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ cordaProductVersion = 5.1.0
# NOTE: update this each time this module contains a breaking change
## NOTE: currently this is a top level revision, so all API versions will line up, but this could be moved to
## a per module property in which case module versions can change independently.
cordaApiRevision = 16
cordaApiRevision = 17

# Main
kotlinVersion = 1.8.21
Expand Down

0 comments on commit 7ea0236

Please sign in to comment.