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.

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

[Feature][Connector-V2] Add prometheus source and sink #7265

Merged
merged 74 commits into from
Oct 29, 2024
Merged

[Feature][Connector-V2] Add prometheus source and sink #7265

merged 74 commits into from
Oct 29, 2024

Conversation

CosmosNi
Copy link
Contributor

@CosmosNi CosmosNi commented Jul 25, 2024 •
edited
Loading

Contribution Checklist

#7201

Purpose of this pull request

supprot prometheus source

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

hailin0
hailin0 reviewed Jul 25, 2024
View reviewed changes
Copy link
Member

@hailin0 hailin0 left a comment

Choose a reason for hiding this comment

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

please add prometheus sink

@hailin0
Copy link
Member

hailin0 commented Jul 27, 2024

@CosmosNi

@hailin0
Copy link
Member

hailin0 commented Aug 8, 2024

hi @CosmosNi
Is there any progress on this task?

@hailin0
Copy link
Member

hailin0 commented Aug 8, 2024 •
edited
Loading

@CosmosNi
If you do not want to add a sink for now, please resolve the conflict and continue to push this PR

@CosmosNi
Copy link
Contributor Author

CosmosNi commented Aug 9, 2024

@hailin0 let me resolve the conflict first.about sink,I'm still looking for related solutions,The native remote writer has relatively little data

CosmosNi added 4 commits August 9, 2024 11:22
@CosmosNi
Merge remote-tracking branch 'upstream/dev' into feature-prometheus-s…
ce29753 
…ource

# Conflicts:
#	config/plugin_config
#	plugin-mapping.properties
#	seatunnel-connectors-v2/pom.xml
#	seatunnel-e2e/seatunnel-connector-v2-e2e/pom.xml
@CosmosNi
Merge remote-tracking branch 'upstream/dev' into feature-prometheus-s…
705e930 
…ource

# Conflicts:
#	plugin-mapping.properties
@CosmosNi
[Feature][prometheus] Support prometheus write
d8b5107
@CosmosNi
[Feature][prometheus] Support prometheus write
b61857f
@CosmosNi CosmosNi changed the title Feature prometheus source Feature prometheus source/sink Aug 13, 2024
@CosmosNi CosmosNi requested a review from hailin0 August 13, 2024 09:42
@CosmosNi CosmosNi mentioned this pull request Sep 18, 2024
Closed
3 tasks
@CosmosNi
Copy link
Contributor Author

CosmosNi commented Sep 19, 2024 •
edited
Loading

@hailin0 @Hisoka-X @corgy-w have time to cc. thanks.Add VictoriaMetrics e2e

@hailin0
Copy link
Member

hailin0 commented Sep 21, 2024

good！

Please fix ci error

@CosmosNi
Copy link
Contributor Author

@hailin0 Please cc if you have time.

@hailin0
Copy link
Member

hailin0 commented Oct 29, 2024

LGTM. Thanks @CosmosNi

@hailin0 hailin0 merged commit dde6f9f into apache:dev Oct 29, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hailin0 hailin0 hailin0 approved these changes

@Hisoka-X Hisoka-X Hisoka-X approved these changes

@corgy-w corgy-w Awaiting requested review from corgy-w

Assignees
No one assigned
Labels
approved CI&CD connectors-v2 document e2e reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature][Prometheus] Support Prometheus connector source/sink
4 participants
@CosmosNi @hailin0 @Hisoka-X @corgy-w