Skip to content
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

wrong subservice in aggegation framework? #2185

Closed
AlvaroVega opened this issue Jul 20, 2022 · 3 comments
Closed

wrong subservice in aggegation framework? #2185

AlvaroVega opened this issue Jul 20, 2022 · 3 comments

Comments

@AlvaroVega
Copy link
Member

Possible bug to analize:

Given a destination in some sinks (SQL, CKAN...)

aggregator.setServicePathForData(events.get(0).getServicePathForData());

but that subsservice could be different in each event of the same destination

@AlvaroVega AlvaroVega changed the title wrong subservice in aggegation framework wrong subservice in aggegation framework? Jul 20, 2022
@AlvaroVega
Copy link
Member Author

AlvaroVega commented Jul 26, 2022

With a name mapping like:

{
 "serviceMappings": [
  {
"originalService": "smartcity",
"newService": "newcity",
"servicePathMappings": [
   {
      "originalServicePath": "/gardens",
      "newServicePath": "/",
      "entityMappings": [
          {
             "originalEntityId": "(.+)",
             "newEntityId": "dumps",
             "originalEntityType": "Device",
             "newEntityType": "Device",
             "attributeMappings": []
          } 
      ]
   },
  {
      "originalServicePath": "/roads",
      "newServicePath": "/",
      "entityMappings": [
          {
             "originalEntityId": "(.+)",
             "newEntityId": "dumps",
             "originalEntityType": "Device",
             "newEntityType": "Device",
             "attributeMappings": []
          } 
      ]
   }
]
}
]
}

@fgalan fgalan added this to the release/2.20.0 milestone Aug 4, 2022
@fgalan
Copy link
Member

fgalan commented Aug 4, 2022

PR #2186

Should this issue be closed now that the PR has been merged?

@fgalan
Copy link
Member

fgalan commented Oct 19, 2022

Cygnus 2.20.0 has been released. Closing issue.

@fgalan fgalan closed this as completed Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants