-
Notifications
You must be signed in to change notification settings - Fork 105
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[OrionKafkaSink] Expand topic name #880
Comments
Some tests are going to be implemented with the PR: Getting
Getting
|
After reviews with this PR we are going to need 3 differents fixes now:
|
Partially implemented in #905 |
Current tests deleted in #915 |
Partially implemented in #926 |
Partially implemented in #929 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently:
data_model=dm-by-service
thentopic_name=<service>
.data_model=dm-by-service-path
thentopic_name=<service-path>
.data_model=dm-by-entity
thentopic_name=<entity>
.data_model=dm-by-attribute
thentopic_name=<attribute>
.This should be instead ("normal" service-path starting with `/' ):
data_model=dm-by-service
thentopic_name=<service>
.data_model=dm-by-service-path
thentopic_name=<service>_<service-path_without_slash>
.data_model=dm-by-entity
thentopic_name=<service>_<service-path_without_slash>_<entityId>_<entityType>
.data_model=dm-by-attribute
thentopic_name=<service>_<service-path_without_slash>_<entityId>_<entityType>_<attrName>_<attrType>
.When the service-path is
/
:data_model=dm-by-service
thentopic_name=<service>
.data_model=dm-by-service-path
thentopic_name=<service>
.data_model=dm-by-entity
thentopic_name=<service>_<entityId>_<entityType>
.data_model=dm-by-attribute
thentopic_name=<service>_<entityId>_<entityType>_<attrName>_<attrType>
.Effort: 1 man day
The text was updated successfully, but these errors were encountered: