Skip to content
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.

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

[INLONG-10287][Agent] Update the Redis Source #10801

Closed
wants to merge 8 commits into from

Conversation

emptyOVO
Copy link
Contributor

@emptyOVO emptyOVO commented Aug 16, 2024

Fixes #10287

Motivation

Update the Redis source for agent

Modifications

Modify TaskProfileDto and RedisTask to accept and parse the configuration.
Add RedisTask, RedisInstance, and RedisSource classes
Provide Unit Test

Verifying this change

6983c4d37da7fa06df06467b6c591dd

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hello @emptyOVO, thank you for submitting a PR to InLong 💖 We will respond as soon as possible ⏳
This seems to be your first PR 🌠 Please be sure to follow our Contribution Guidelines.
If you have any questions in the meantime, you can also ask us on the InLong Discussions 🔍

@dockerzhang dockerzhang requested a review from haifxu August 19, 2024 12:12
@emptyOVO emptyOVO requested a review from justinwwhuang August 19, 2024 12:19
@emptyOVO emptyOVO closed this Aug 21, 2024
@emptyOVO emptyOVO reopened this Aug 21, 2024
@emptyOVO emptyOVO closed this Aug 21, 2024
@emptyOVO emptyOVO reopened this Aug 21, 2024
@dockerzhang
Copy link
Contributor

@emptyOVO please click resolve if you fix the comment, and you do not need to open/close the PR again and again.

@emptyOVO
Copy link
Contributor Author

@emptyOVO please click resolve if you fix the comment, and you do not need to open/close the PR again and again.

got it

…bscribed a key operation, it filters the key we want and jedis is used to obtain the key data
…plicator to subscribe event and filtering data by command
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Improve][Agent] Update the Redis Source
3 participants