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

[Improve][Sls] Add sls sink connector、e2e、doc #7830

Merged
merged 5 commits into from
Oct 14, 2024
Merged

Conversation

XenosK
Copy link
Contributor

@XenosK XenosK commented Oct 14, 2024

Thank you for contributing to SeaTunnel! Please make sure that your code changes
are covered with tests. And in case of new features or big changes
remember to adjust the documentation.

Feel free to ping committers for the review!

Contribution Checklist

  • Make sure that the pull request corresponds to a GITHUB issue.
  • Name the pull request in the form "[Feature] [component] Title of the pull request", where Feature can be replaced by Hotfix, Bug, etc.
  • Minor fixes should be named following this pattern: [hotfix] [docs] Fix typo in README.md doc.
    -->

Purpose of this pull request

This pull request adds sink connector for sls

Does this PR introduce any user-facing change?

no

How was this patch tested?

e2e run locally

Check list

@XenosK
Copy link
Contributor Author

XenosK commented Oct 14, 2024

截屏2024-10-14 13 39 13
Aliyun sls need a personally account, screenshot of successfully run ete

Copy link
Member

@Hisoka-X Hisoka-X left a comment

Choose a reason for hiding this comment

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

Overall LGTM. Thanks @XenosK

@hailin0 hailin0 merged commit 048c47d into apache:dev Oct 14, 2024
7 checks passed
CosmosNi pushed a commit to CosmosNi/seatunnel that referenced this pull request Oct 15, 2024
Co-authored-by: XenosK <xiangqiang.kong@sunmi.com>
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.

3 participants