search

snowplow / snowbridge

For replicating streams across clouds, accounts and regions
Other
14 stars 8 forks source link

Cross event transformation #331

Closed colmsnowplow closed 2 months ago

colmsnowplow commented 2 months ago

This isn't ready for develop, but the refactoring part of things is basically done, so it feels like a good time to get a review and split up the rest of the work to separate it from the complex bits.

There are still a lot of TODO notes etc in the code, and the work to migrate the targets to the new structure, fix and add tests etc. is yet to happen.

But at this point this PR is in service of getting a review of the overall structure. we can treat the feature branch as a WIP, and get it ready for a PR to develop bit by bit.

colmsnowplow commented 2 months ago

@jbeemster should be pointers now and @pondzix redid the dir structure :)

colmsnowplow commented 2 months ago

Closing in favour of https://github.com/snowplow/snowbridge/pull/333, which allows us to avoid the more complex aspects, at least until we reach the need for the features/functionality that it would facilitate.

Thanks for the feedback @adatzer and @pondzix - wouldn't have landed on the better path without that input