Skip to content

docs: redis sink docs#40

Merged
lavkesh merged 8 commits intomainfrom
docs/redis-sink-docs
Sep 21, 2022
Merged

docs: redis sink docs#40
lavkesh merged 8 commits intomainfrom
docs/redis-sink-docs

Conversation

@sumitaich1998
Copy link
Copy Markdown
Contributor

No description provided.

@sumitaich1998 sumitaich1998 marked this pull request as ready for review September 20, 2022 06:23
Comment thread docs/sinks/redis.md Outdated
Comment thread docs/sinks/redis.md Outdated
Comment thread docs/sinks/redis.md Outdated
Comment thread docs/sinks/redis.md Outdated
# Redis Sink

### Input Schema
Redis sink in Depot only supports Protobuf and JSON schema as of now. Support for Avro format is planned for a future release of Depot.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we don't need to put formats here.
because it's a not part of sink. It is independent. Once we have parsers, all the sinks will support the format and no code change will be needed

Comment thread docs/sinks/redis.md Outdated
### Data Types
Redis sink can be created in 3 different modes based on the value of [`SINK_REDIS_DATA_TYPE`](../reference/configuration/redis.md#sink_redis_data_type): HashSet, KeyValue or List
- `Hashset`: For each message, an entry of the format `key : field : value` is generated and pushed to Redis. Field and value are generated on the basis of the config [`SINK_REDIS_HASHSET_FIELD_TO_COLUMN_MAPPING`](../reference/configuration/redis.md#sink_redis_hashset_field_to_column_mapping)
- `List`: For each message entry of the format `key : value` is generated and pushed to Redis. Value is fetched for the proto index provided in the config [`SINK_REDIS_LIST_DATA_PROTO_INDEX`](../reference/configuration/redis.md#sink_redis_list_data_proto_index) or the Proto field name provided in the config [`SINK_REDIS_LIST_DATA_FIELD_NAME`](../reference/configuration/redis.md#sink_redis_list_data_field_name)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah. we need to delete SINK_REDIS_LIST_DATA_PROTO_INDEX it's not required.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SINK_REDIS_LIST_DATA_FIELD_NAME this is the config.

@lavkesh lavkesh merged commit c087c86 into main Sep 21, 2022
@lavkesh lavkesh deleted the docs/redis-sink-docs branch September 21, 2022 10:00
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

Successfully merging this pull request may close these issues.

2 participants