Skip to content
This repository has been archived by the owner on Dec 7, 2022. It is now read-only.

Adding kafka to msk bridge for ConceptAnnotations topic. #97

Merged

Conversation

martin-stanchev
Copy link
Contributor

@martin-stanchev martin-stanchev commented Jun 27, 2022

Description

What

Part of MSK migration.

Why

https://financialtimes.atlassian.net/browse/UPPSF-3287

Anything, in particular, you'd like to highlight to reviewers

Mention here sections of code which you would like reviewers to pay extra attention to .E.g

Would appreciate a second pair of eyes on the test
I am not quite sure how this bit works
Is there a better library for doing x

Scope and particulars of this PR (Please tick all that apply)

  • Tech hygiene (dependency updating & other tech debt)
  • Bug fix
  • Feature
  • Documentation
  • Breaking change
  • Minor change (e.g. fixing a typo, adding config)

This Pull Request follows the rules described in our Pull Requests Guide

@martin-stanchev martin-stanchev requested a review from a team as a code owner June 27, 2022 13:58
@martin-stanchev martin-stanchev requested a review from a team June 27, 2022 13:59
@dtvalk-ov dtvalk-ov requested a review from a team June 28, 2022 07:34
@martin-stanchev martin-stanchev merged commit 5477638 into master Jun 29, 2022
@martin-stanchev martin-stanchev deleted the feature/UPPSF-3287-concept-annotations-bridge branch June 29, 2022 07:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants