SR: Processor encodewithschema #1098

lovromazgon · 2023-06-30T17:10:34Z

⚠️ This PR is based on top of #1097.

This PR adds a processor encodewithschema as described here.

Fixes #1044

I have followed the Code Guidelines.
There is no other pull request for the same update/change.
I have written unit tests. (if I get the time I'll add some later, did manually test though)
I have made sure that the PR is of reasonable size and can be easily reviewed.

pkg/processor/procbuiltin/encodewithschema.go

pkg/processor/procbuiltin/decodewithschema.go

hariso

LGTM! 🥇

lovromazgon requested a review from a team as a code owner June 30, 2023 17:10

nassor reviewed Jul 12, 2023

View reviewed changes

pkg/processor/procbuiltin/encodewithschema.go Show resolved Hide resolved

lovromazgon force-pushed the lovro/sr-decodewithschema branch from 051466b to daad555 Compare July 13, 2023 14:18

lovromazgon force-pushed the lovro/sr-encodewithschema branch from 6d06eb6 to 0191cc3 Compare July 13, 2023 14:20

lovromazgon mentioned this pull request Jul 13, 2023

SR: Processor decodewithschema #1097

Merged

4 tasks

hariso reviewed Jul 14, 2023

View reviewed changes

pkg/processor/procbuiltin/decodewithschema.go Show resolved Hide resolved

hariso approved these changes Jul 14, 2023

View reviewed changes

lovromazgon force-pushed the lovro/sr-decodewithschema branch from daad555 to 4cd70ed Compare July 14, 2023 13:12

Base automatically changed from lovro/sr-decodewithschema to main July 14, 2023 13:15

lovromazgon added 3 commits July 14, 2023 15:17

implement processor encodewithschema

3b8bdf4

add processor documentations

5350aa8

rename strategy to preRegistered

e88248e

lovromazgon force-pushed the lovro/sr-encodewithschema branch from 9feb194 to e88248e Compare July 14, 2023 13:18

lovromazgon merged commit 00b362f into main Jul 14, 2023

lovromazgon deleted the lovro/sr-encodewithschema branch July 14, 2023 13:25

Provide feedback