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

[Feature][Core] Support event listener for job #6419

Merged
merged 12 commits into from
Mar 28, 2024

Conversation

hailin0
Copy link
Member

@hailin0 hailin0 commented Feb 29, 2024

Purpose of this pull request

Support event listener for job:

  • zeta
  • flink
  • spark

Does this PR introduce any user-facing change?

Zeta seatunnel.yaml

seatunnel:
  engine:
    event-report-http:
      url: http://example.com/event/report

Flink/Spark
SPI load EventListener impl class from classpath

How was this patch tested?

Added

Check list

@hailin0 hailin0 force-pushed the dev-support-event-listener branch 2 times, most recently from 2cd76bf to 73ac99e Compare February 29, 2024 07:51
…c/test/java/org/apache/seatunnel/e2e/connector/fake/FakeSourceToConsoleWithEventReportIT.java
@hailin0
Copy link
Member Author

hailin0 commented Mar 2, 2024

img_v3_028j_ad0224ef-9565-4aa7-82ae-f9d407d3468g

@hailin0
Copy link
Member Author

hailin0 commented Mar 2, 2024

image

@hailin0
Copy link
Member Author

hailin0 commented Mar 2, 2024

image

@hailin0 hailin0 added the core SeaTunnel core module label Mar 15, 2024
@hailin0 hailin0 added this to the 2.3.5 milestone Mar 25, 2024
Copy link
Member

@Carl-Zhou-CN Carl-Zhou-CN left a comment

Choose a reason for hiding this comment

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

+1

@hailin0 hailin0 merged commit 831d002 into apache:dev Mar 28, 2024
6 checks passed
@liunaijie liunaijie mentioned this pull request May 11, 2024
3 tasks
chaorongzhi pushed a commit to chaorongzhi/seatunnel that referenced this pull request Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants