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.

Sign up for GitHub

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

Jump to bottom

redis sink: complex protobuf data type values not getting synced properly #61

Open

kevinbheda opened this issue Nov 17, 2022 · 0 comments

Assignees

Labels

Contributor

kevinbheda commented Nov 17, 2022

Description
redis sink: complex protobuf data type values not getting synced properly
Steps To Reproduce

REDIS SINK
With config INPUT_SCHEMA_PROTO_TO_COLUMN_MAPPING: {"2":"topics"}
With config SINK_REDIS_KEY_TEMPLATE: patch-subscription:%s,customer_name
With protobuf message as
message TestBookingLogMessage { string customer_name = 1; message TopicMetadata { string topic = 1; int32 qos = 2; } repeated TopicMetadata topics = 2;

Expected behavior
""[{"topic":"world/pns/1/com.hello.owl/701483858/3ac6a577fbd334b4","qos":1}]""

Additional context
"[topic: "world/pns/1/com.hello.owl/701483858/3ac6a577fbd334b4"\nqos: 1\n]"

Proposed Fix.
Use JsonPrinter in ProtoOdpfParsedMessage.getFieldByName()

The text was updated successfully, but these errors were encountered:

kevinbheda self-assigned this

kevinbheda added bug redis-sink labels

kevinbheda mentioned this issue

fix: redis sink handle complex data types #62

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment